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 <ottw...@gmail.com> 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 > > > > > >
