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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.