<table><tr><td style="">dfaure requested changes to this revision.<br />dfaure added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18915">View Revision</a></tr></table><br /><div><div><p>Good catch, but then this is a bug in QMimeDatabase::suffixForFileName.</p>

<p>I just submitted a fix at <a href="https://codereview.qt-project.org/c/qt/qtbase/+/274548" class="remarkup-link" target="_blank" rel="noreferrer">https://codereview.qt-project.org/c/qt/qtbase/+/274548</a> -- aiming at the 5.14 branch.</p>

<p>So this commit is still valid of course, but it would be good to use #if QT_VERSION < QT_VERSION_CHECK(5, 14, 0) around the workaround, so that it goes away once we use a Qt version with the fix.<br />
AFAICS the workaround is only the use of "extensionLen", though (that's what should be ifdef'ed). This commit doesn't only fix casing, it also fixes the case of unknown extensions. That's valid and can be kept, no matter which Qt version we're using.</p>

<p>I don't understand the comment about right-to-left languages in the commit log; this isn't using any locale-dependent APIs, is it?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18915">https://phabricator.kde.org/D18915</a></div></div><br /><div><strong>To: </strong>cfoster, Dolphin, Frameworks, abalaji, dfaure<br /><strong>Cc: </strong>cfeck, bruns, ngraham, elvisangelaccio, chinmoyr, kde-frameworks-devel, LeGast00n, GB_2, michaelh<br /></div>