Clean up includes
ClosedPublic

Authored by zzag on Aug 31 2018, 6:34 PM.

Details

Reviewers
davidedmundson
Group Reviewers
KWin
Commits
R108:624a453109c5: Clean up includes
Summary
  • effects.h includes client.h and forward declares Client, both at the same time. Thus, delete the include;
Test Plan

Compiles.

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.Aug 31 2018, 6:34 PM
Restricted Application added a project: KWin. · View Herald TranscriptAug 31 2018, 6:34 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Aug 31 2018, 6:34 PM
zzag added a comment.Aug 31 2018, 6:40 PM

Not sure if "a leftover after 3f5bf65a9ec8fa413ca59dd07df267c8b765329c" is a correct term. But the point is the blur effect should not include kwin's internal headers.

davidedmundson accepted this revision.Aug 31 2018, 7:02 PM
This revision is now accepted and ready to land.Aug 31 2018, 7:02 PM
This revision was automatically updated to reflect the committed changes.