refactor: Minimize use of geom in Toplevel subclasses
ClosedPublic

Authored by zzag on Aug 10 2019, 2:45 PM.

Details

Summary

This change makes easier to refactor geometry handling in the future.
The main motivation for avoiding using geom directly is to make code
more readable and ensure that the geometry is updated only through
designated methods, e.g. setGeometry, plainResize, etc.

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 10 2019, 2:45 PM
Restricted Application added a project: KWin. · View Herald TranscriptAug 10 2019, 2:45 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Aug 10 2019, 2:45 PM
zzag updated this revision to Diff 63487.Aug 10 2019, 2:48 PM

Capture only clientGeometry.

romangg accepted this revision.Aug 10 2019, 3:28 PM
romangg added a subscriber: romangg.

That's good. I hate this geom member variable without m_.

This revision is now accepted and ready to land.Aug 10 2019, 3:28 PM
This revision was automatically updated to reflect the committed changes.