[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
Branch
effects-slide-fix-coding-style
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1045
Build 1058: arc lint + arc unit
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.