- use NET::KeepAbove instead of NET::StaysOnTop in code
- deduplicate code handling the value of NET::KeepAbove/NET::StaysOnTop
- remove mention of NET::StaysOnTop in API dox where just alias
Details
Details
- Reviewers
zzag - Group Reviewers
KWin - Commits
- R278:c747c71d7146: Make less use of deprecated enum alias NET::StaysOnTop
Diff Detail
Diff Detail
- Repository
- R278 KWindowSystem
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Found while experimenting with code for T11490.
This patch would later allow to do a simple
#if KWINDOWSYSTEM_ENABLE_DEPRECATED_SINCE(5, 0) /** @deprecated This is an obsolete name for KeepAbove. **/ StaysOnTop = KeepAbove, // NOT STANDARD #endif
to hide the enum alias on demand, without breaking the build of kwindowsystem itself
autotests/netwininfotestwm.cpp | ||
---|---|---|
448–449 | We can remove this row now. |