Kdiff3 in kdereview

Albert Astals Cid aacid at kde.org
Thu Aug 23 23:07:49 BST 2018

El dimarts, 7 d’agost de 2018, a les 14:59:50 CEST, Michael Reeves va escriure:
> Kdiff3 has moved to review in preparation for possible release testing. I
> am currently working towards having auto testing but the code needs major
> refactoring to make this possible. Specifically it is not well modularized.
> The purpose of this review is to get feedback on issues that need to be
> addressed before moving out of playground.

Have you seen there's 4 wrong connect signals on startup?

When trying to compare any two files i hit this assert

        if(m_lmppData.m_vSize < m_normalData.m_vSize)
            //This a bug that needs fixed elsewhere not hacked around
            Q_ASSERT(m_lmppData.m_vSize == m_normalData.m_vSize);

Which i do not understand what it is trying to do, i mean you just checked that they are different and the on the next line assert they are not different?


More information about the kde-core-devel mailing list