Differential D25738

Rename Toplevel::geom to Toplevel::m_frameGeometry
ClosedPublic
Actions

Authored by zzag on Dec 4 2019, 1:36 PM.

Tags

Subscribers

Details

Reviewers

Group Reviewers

Commits

R108:d39485553630: Rename Toplevel::geom to Toplevel::m_frameGeometry

Summary

The new name better reflects what Toplevel::geom is.

Test Plan

Compiles, tests still pass.

Diff Detail

Repository

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.Dec 4 2019, 1:36 PM

Restricted Application added a project: KWin. · View Herald TranscriptDec 4 2019, 1:36 PM

Restricted Application added a subscriber: kwin. · View Herald Transcript

zzag requested review of this revision.Dec 4 2019, 1:36 PM

Harbormaster completed remote builds in B19485: Diff 70900.Dec 4 2019, 1:36 PM

davidedmundson accepted this revision.Dec 4 2019, 1:44 PM

This revision is now accepted and ready to land.Dec 4 2019, 1:44 PM

Closed by commit R108:d39485553630: Rename Toplevel::geom to Toplevel::m_frameGeometry (authored by zzag). · Explain WhyDec 4 2019, 1:47 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			events.cpp (8 lines)
M			geometry.cpp (8 lines)
M			internal_client.cpp (10 lines)
M			toplevel.h (16 lines)
M			toplevel.cpp (2 lines)
M			unmanaged.cpp (4 lines)
M			x11client.cpp (4 lines)
M			xdgshellclient.cpp (14 lines)

Diff 70902

events.cpp

Loading...

geometry.cpp

Loading...

internal_client.cpp

Loading...

toplevel.h

Loading...

toplevel.cpp

Loading...

unmanaged.cpp

Loading...

x11client.cpp

Loading...

xdgshellclient.cpp

Loading...