Re: Review Request 117777: Make the KSelectionProxyModel unit test pass
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/#review56594 --- Would it be a good idea to start by reverting https://git.reviewboard.kde.org/r/116096/ ? - Stephen Kelly On April 25, 2014, 7:18 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/ --- (Updated April 25, 2014, 7:18 p.m.) Review request for KDE Frameworks and Stephen Kelly. Repository: kitemmodels Description --- Make the KSelectionProxyModel unit test pass It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits rowsMoved. This could be used to optimize things, but for now we just act the same way in either case (rebuilding the mapping). Add a pedantic mode to modeltest. Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703 This was lost when I updated ModelTest to Qt5. Re-apply Stephen Kelly's modeltest fixes These were lost when I replaced ModelTest with the version from Qt 5. Diffs - autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 Diff: https://git.reviewboard.kde.org/r/11/diff/ Testing --- Unit tests now pass. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117777: Make the KSelectionProxyModel unit test pass
On April 26, 2014, 8:51 a.m., Stephen Kelly wrote: Would it be a good idea to start by reverting https://git.reviewboard.kde.org/r/116096/ ? Alex Merry wrote: So we have no unit tests? I don't really see how that would help (except in the strict sense of no unit tests = no failing unit tests). Stephen Kelly wrote: I meant considering the problem again as it was before that patch - namely doing a proper merge of the two versions. Is that what this patch does? Is it a full merge? Yes, the first two commits merge the two versions (by reapplying all the changes you made on top of the new one). The third commit alters KSelectionProxyModel. Would it be less confusing as two reviews? - Alex --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/#review56594 --- On April 25, 2014, 7:18 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/ --- (Updated April 25, 2014, 7:18 p.m.) Review request for KDE Frameworks and Stephen Kelly. Repository: kitemmodels Description --- Make the KSelectionProxyModel unit test pass It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits rowsMoved. This could be used to optimize things, but for now we just act the same way in either case (rebuilding the mapping). Add a pedantic mode to modeltest. Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703 This was lost when I updated ModelTest to Qt5. Re-apply Stephen Kelly's modeltest fixes These were lost when I replaced ModelTest with the version from Qt 5. Diffs - autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 Diff: https://git.reviewboard.kde.org/r/11/diff/ Testing --- Unit tests now pass. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117777: Make the KSelectionProxyModel unit test pass
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/#review56629 --- Ship it! Ship It! - Kevin Ottens On April 25, 2014, 7:18 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/ --- (Updated April 25, 2014, 7:18 p.m.) Review request for KDE Frameworks and Stephen Kelly. Repository: kitemmodels Description --- Make the KSelectionProxyModel unit test pass It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits rowsMoved. This could be used to optimize things, but for now we just act the same way in either case (rebuilding the mapping). Add a pedantic mode to modeltest. Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703 This was lost when I updated ModelTest to Qt5. Re-apply Stephen Kelly's modeltest fixes These were lost when I replaced ModelTest with the version from Qt 5. Diffs - autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 Diff: https://git.reviewboard.kde.org/r/11/diff/ Testing --- Unit tests now pass. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117777: Make the KSelectionProxyModel unit test pass
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/#review56633 --- This review has been submitted with commit 2fc7aeab836b0e40360024bf57630fe2cfb66145 by Alex Merry to branch master. - Commit Hook On April 25, 2014, 7:18 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/ --- (Updated April 25, 2014, 7:18 p.m.) Review request for KDE Frameworks and Stephen Kelly. Repository: kitemmodels Description --- Make the KSelectionProxyModel unit test pass It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits rowsMoved. This could be used to optimize things, but for now we just act the same way in either case (rebuilding the mapping). Add a pedantic mode to modeltest. Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703 This was lost when I updated ModelTest to Qt5. Re-apply Stephen Kelly's modeltest fixes These were lost when I replaced ModelTest with the version from Qt 5. Diffs - autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 Diff: https://git.reviewboard.kde.org/r/11/diff/ Testing --- Unit tests now pass. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117777: Make the KSelectionProxyModel unit test pass
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/#review56632 --- This review has been submitted with commit 7cd8252d2c662c6db4a90788a64fec8297c6c378 by Alex Merry on behalf of Stephen Kelly to branch master. - Commit Hook On April 25, 2014, 7:18 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/ --- (Updated April 25, 2014, 7:18 p.m.) Review request for KDE Frameworks and Stephen Kelly. Repository: kitemmodels Description --- Make the KSelectionProxyModel unit test pass It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits rowsMoved. This could be used to optimize things, but for now we just act the same way in either case (rebuilding the mapping). Add a pedantic mode to modeltest. Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703 This was lost when I updated ModelTest to Qt5. Re-apply Stephen Kelly's modeltest fixes These were lost when I replaced ModelTest with the version from Qt 5. Diffs - autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 Diff: https://git.reviewboard.kde.org/r/11/diff/ Testing --- Unit tests now pass. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117777: Make the KSelectionProxyModel unit test pass
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/#review56631 --- This review has been submitted with commit fd6d554d8af7fe74d6c2e911a19c95a3b39268ee by Alex Merry to branch master. - Commit Hook On April 25, 2014, 7:18 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/ --- (Updated April 25, 2014, 7:18 p.m.) Review request for KDE Frameworks and Stephen Kelly. Repository: kitemmodels Description --- Make the KSelectionProxyModel unit test pass It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits rowsMoved. This could be used to optimize things, but for now we just act the same way in either case (rebuilding the mapping). Add a pedantic mode to modeltest. Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703 This was lost when I updated ModelTest to Qt5. Re-apply Stephen Kelly's modeltest fixes These were lost when I replaced ModelTest with the version from Qt 5. Diffs - autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 Diff: https://git.reviewboard.kde.org/r/11/diff/ Testing --- Unit tests now pass. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 117777: Make the KSelectionProxyModel unit test pass
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/11/ --- Review request for KDE Frameworks and Stephen Kelly. Repository: kitemmodels Description --- Make the KSelectionProxyModel unit test pass It looks like Qt4 emitted layoutChanged in circumstances where Qt5 emits rowsMoved. This could be used to optimize things, but for now we just act the same way in either case (rebuilding the mapping). Add a pedantic mode to modeltest. Cherry-pick of ffd732966d1f2c632d811e8fc378a95aa1456703 This was lost when I updated ModelTest to Qt5. Re-apply Stephen Kelly's modeltest fixes These were lost when I replaced ModelTest with the version from Qt 5. Diffs - autotests/CMakeLists.txt 4281fec2de1124162af623cf999db73eff705684 autotests/proxymodeltestsuite/modeltest.h 4b80850d51ca068fb8ffb0a704a9f938b4dacfa9 autotests/proxymodeltestsuite/modeltest.cpp 0928a3a4d78d10fcae4a5671e91e836e9fde429b src/kselectionproxymodel.cpp ea97f7a197f9181745f2979c2f7b2e5e1f0015f3 Diff: https://git.reviewboard.kde.org/r/11/diff/ Testing --- Unit tests now pass. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel