Web lists-archives.com

Re: Q_ASSERT(!FalseSecurity)




Good pickup - I'm not sure of the answer but my interpretation of it is the same as yours.

Here's a StackOverflow thread that might be good reference for anyone else that wants to chime in:
https://stackoverflow.com/questions/12573230/q-assert-release-build-semantics#12573446

Cheers!

On Sat, Mar 10, 2018 at 7:53 PM, Michael Heidelbach <ottwolt@xxxxxxxxx> wrote:

Hi!

Am I getting something wrong? Or is

"Q_ASSERT(m_writeTrans);

m_writeTrans->commit();"

providing false security?

Shouldn't it better be

"Q_ASSERT(m_writeTrans);

if (m_writeTrans) {

    m_writeTrans->commit();  ?

Baloo's code is full of Q_ASSERTs that have no corresponding if-guard.

To my understanding those would slip through in release-builds. Am I wrong?


Please enlighten me,

Michael