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





Reply via email to