[effects/slide] Fix coding style
ClosedPublic

Authored by zzag on Jul 21 2018, 8:44 AM.

Details

Summary

Kdelibs coding style states the following about whitespace placement:

For pointers or references, use a single space before '*' or '&', but not after

Source: https://community.kde.org/Policies/Kdelibs_Coding_Style#Whitespace

Also, putting a whitespace after ! is not common in KWin.

Test Plan

Switched between virtual desktops.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.Jul 21 2018, 8:44 AM
Restricted Application added a project: KWin. · View Herald TranscriptJul 21 2018, 8:44 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Jul 21 2018, 8:44 AM
zzag added a comment.Jul 21 2018, 8:45 AM

In my defense: I didn't know about existence of Kdelibs coding style when I was working on the Sldie effect.

davidedmundson accepted this revision.Jul 21 2018, 11:32 AM
davidedmundson added a subscriber: davidedmundson.

In my defense: I didn't know about existence of Kdelibs coding style when I was working on the Sldie effect.

Heh, no problem. Personally I don't really care about a few spaces as long as things are readable. Ship it!

This revision is now accepted and ready to land.Jul 21 2018, 11:32 AM
This revision was automatically updated to reflect the committed changes.