D23605: Switch to Q_ASSERT

Vlad Zagorodniy noreply at phabricator.kde.org
Sat Aug 31 15:50:21 BST 2019


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190831/7004b41f/attachment.html>


More information about the kwin mailing list