fix(kwayland): remove code with side-effects from Q_ASSERT
ClosedPublic

Authored by dvratil on Feb 21 2020, 7:41 AM.

Details

Summary

BUG: 413892
FIXED-IN: 5.18.2

Diff Detail

Repository
R110 KScreen Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dvratil created this revision.Feb 21 2020, 7:41 AM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 21 2020, 7:41 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
dvratil requested review of this revision.Feb 21 2020, 7:41 AM
dvratil updated this revision to Diff 76080.Feb 21 2020, 7:43 AM
  • Fix typo
dvratil updated this revision to Diff 76082.Feb 21 2020, 9:18 AM
  • Add bug reference
dvratil edited the summary of this revision. (Show Details)Feb 21 2020, 9:21 AM
davidedmundson accepted this revision.Feb 21 2020, 10:05 AM
This revision is now accepted and ready to land.Feb 21 2020, 10:05 AM
meven accepted this revision.Feb 21 2020, 11:13 AM
meven added a subscriber: meven.

Nice !

This revision was automatically updated to reflect the committed changes.

It would be nice if I get at least 24 hours to review patches in KScreen/libkscreen. Thanks for the patch.