projects / profile / ivi / qtdeclarative.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 60204c8) | patch
Remove Q_ASSERT calls that wrap side-effecting code.
authorJason McDonald <jason.mcdonald@nokia.com>
Fri, 15 Apr 2011 04:15:49 +0000 (14:15 +1000)
committerRohan McGovern <rohan.mcgovern@nokia.com>
Wed, 18 May 2011 00:59:10 +0000 (10:59 +1000)
commit891b216c319560e823a3e05e9d6aa8e9e0128426
treebbc8e6926783bdde6e822b061072f1bfcce26ea4tree | snapshot
parent60204c8325b0738823c81993fcdc4d2a5409a7a0commit | diff
Remove Q_ASSERT calls that wrap side-effecting code.

In release mode builds, code inside Q_ASSERT macros is not executed, so
putting code with side-effects inside Q_ASSERT is not a good idea.

Task-number: QTBUG-17582
Change-Id: I1a5d8ccce666ee7b7f120bf9cbb49e30dac9add4
Reviewed-by: Rohan McGovern
(cherry picked from commit 1750a4351a8125756bf38a53d6d8312993c7d85e)
tests/auto/declarative/qmlvisual/tst_qmlvisual.cpp diff | blob | history
Domain: UI Framework / Uncategorized;