Switch to Q_ASSERT in order to make code a bit more consistent. We have
places where 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.
Details
Details
- Reviewers
davidedmundson - Group Reviewers
KWin - Commits
- R108:7a3722b4f5a3: Switch to Q_ASSERT
Compiles.
Diff Detail
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.
Comment Actions
From my side it's ok to go in. @davidedmundson any objections against cleaning house?
libkwineffects/kwineffects.cpp | ||
---|---|---|
938 | heh, if we're going to spend time porting these, we may as well have gone to if (kwinLoggingCategory.isEnabled) and used the modern runtime version |