D23089: KWindowSystem: remove unused QWidget includes

2019-08-11 Thread Vlad Zagorodniy
zzag added a comment. In D23089#510308 , @dfaure wrote: > That's for the benefit of the kde-frameworks-devel mailing list where it's all a bit mixed up otherwise... Oh, I didn't know that. Thank you for the clarification. :-) REPOSITORY

D23089: KWindowSystem: remove unused QWidget includes

2019-08-11 Thread David Faure
dfaure added a comment. In D23089#510276 , @zzag wrote: > "KWindowSystem:" can be omitted in the subject line. That's for the benefit of the kde-frameworks-devel mailing list where it's all a bit mixed up otherwise... REPOSITORY R278 K

D23089: KWindowSystem: remove unused QWidget includes

2019-08-11 Thread David Faure
dfaure closed this revision. REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D23089 To: dfaure, broulik, davidedmundson, zzag Cc: kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D23089: KWindowSystem: remove unused QWidget includes

2019-08-11 Thread Vlad Zagorodniy
zzag accepted this revision. zzag added a comment. This revision is now accepted and ready to land. "KWindowSystem:" can be omitted in the subject line. INLINE COMMENTS > kwindowsystem.cpp:28 > > +#include > #include coding stykle nitpick: Each section is sorted alphabetically. :-) REP

D23089: KWindowSystem: remove unused QWidget includes

2019-08-11 Thread David Faure
dfaure created this revision. dfaure added reviewers: broulik, davidedmundson, zzag. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. dfaure requested review of this revision. REVISION SUMMARY This also gets rid of KWINDOWSYSTEM_NO_QWIDGET TEST PLAN Builds