D23605: Switch to Q_ASSERT
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag created this revision. zzag added a reviewer: KWin. Herald added a project: KWin. Herald added a subscriber: kwin. zzag requested review of this revision. REVISION SUMMARY Switch to Q_ASSERT in order to make code a bit more consistent. We have places were both assert and Q_ASSERT are used next to each other. Also, distributions like Ubuntu don't strip away assert(), let's hope that things are a bit different with Q_ASSERT. TEST PLAN Compiles. REPOSITORY R108 KWin BRANCH switch-to-qassert REVISION DETAIL https://phabricator.kde.org/D23605 AFFECTED FILES abstract_client.cpp activation.cpp client.cpp composite.cpp deleted.cpp effects.cpp effects/lookingglass/lookingglass.cpp effects/presentwindows/presentwindows.cpp geometry.cpp group.cpp kcmkwin/kwinrules/ruleslist.cpp kcmkwin/kwinrules/ruleswidget.cpp layers.cpp libkwineffects/kwineffects.cpp libkwineffects/kwineffects.h libkwineffects/kwinglutils.cpp manage.cpp placement.cpp platformsupport/scenes/opengl/abstract_egl_backend.cpp plugins/platforms/hwcomposer/hwcomposer_backend.cpp plugins/platforms/x11/standalone/glxbackend.cpp plugins/platforms/x11/standalone/overlaywindow_x11.cpp plugins/scenes/opengl/scene_opengl.cpp scene.cpp tabbox/tabbox.cpp toplevel.cpp toplevel.h useractions.cpp utils.cpp workspace.cpp To: zzag, #kwin Cc: kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart