Differential D13355

[effects/minimizeanimation] Include QVector2D by its class name
AbandonedPublic
Actions

Authored by zzag on Jun 5 2018, 9:52 AM.

Details

Reviewers
broulik
Group Reviewers
KWin
Summary

See Kdelibs coding style: https://community.kde.org/Policies/Kdelibs_Coding_Style#Qt_Includes

Diff Detail

Repository
R108 KWin
Branch
effects-minimizeanimation-kdelibs-include
Lint
No Linters Available
Unit
No Unit Test Coverage
zzag created this revision.Jun 5 2018, 9:52 AM
Restricted Application added a project: KWin. · View Herald TranscriptJun 5 2018, 9:52 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Jun 5 2018, 9:52 AM
broulik accepted this revision.Jun 5 2018, 9:57 AM
This revision is now accepted and ready to land.Jun 5 2018, 9:57 AM
davidedmundson added a subscriber: davidedmundson.Jun 5 2018, 9:58 AM

There's a script to port all of them somewhere.
I'll find that rather than have a billion one line reviews.

zzag abandoned this revision.Jun 5 2018, 10:02 AM
In D13355#274137, @davidedmundson wrote:

There's a script to port all of them somewhere.
I'll find that rather than have a billion one line reviews.

OK. I guess that would supersede this diff so I abandon it.

Revision Contents
Changeset List

Diff 35596

View Options

effects/minimizeanimation/minimizeanimation.cpp

Loading...