dfaure added inline comments. INLINE COMMENTS
> elvisangelaccio wrote in kfiledialog_unittest.cpp:116 > No, selectedMimeTypeFilter() is only in 5.9 so that wouldn't compile Oh I see. The QCOMPARE is just so that a failure happens. Pretty pointless, it's not testing anything from Qt. I would just remove the whole #else block. The point of QEXPECT_FAIL is to either mark a TODO (but you fixed it already) or detect unexpected passes, which can't happen here ;-) REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL https://phabricator.kde.org/D5446 To: elvisangelaccio, #plasma, dfaure Cc: plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol