Differential D29104

KCM KWinScreenEdge fix build issue
ClosedPublic
Actions

Authored by crossi on Apr 22 2020, 4:35 PM.

Details

Reviewers
ngraham
zzag
Group Reviewers
KWin
Commits
R108:db9d7a7f5d9c: KCM KWinScreenEdge fix build issue
Summary

Remove Q_ASSERT() and add a None value to Monitor::Edge enum to convert ElectricBorder NONE or COUNT

Test Plan

Fix kwin build

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.
crossi created this revision.Apr 22 2020, 4:35 PM
Restricted Application added a project: KWin. · View Herald TranscriptApr 22 2020, 4:35 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
crossi requested review of this revision.Apr 22 2020, 4:35 PM
ngraham accepted this revision.Apr 22 2020, 4:48 PM
This revision is now accepted and ready to land.Apr 22 2020, 4:48 PM
crossi updated this revision to Diff 80915.Apr 22 2020, 4:52 PM

Update code comment accordingly

zzag accepted this revision.Apr 23 2020, 5:10 AM
Closed by commit R108:db9d7a7f5d9c: KCM KWinScreenEdge fix build issue (authored by crossi). · Explain WhyApr 23 2020, 6:48 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 80957

View Options

kcmkwin/kwinscreenedges/kwinscreenedge.h

Loading...
View Options

kcmkwin/kwinscreenedges/kwinscreenedge.cpp

Loading...
View Options

kcmkwin/kwinscreenedges/monitor.h

Loading...