<table><tr><td style="">meven added inline comments.
</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/D26690">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D26690#inline-151478">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dfaure</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kmimeassociations.cpp:104</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">More precisely, it assumes that a single mimetype entry doesn't have more than 25 desktop files associated with it. And even then, what would happen is that the ordering is wrong after the first 25. I'd say after 25 you start to not really care about the order....</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The first 25 with the highest preferences cannot be affected but then they can have the same base reference as the next most category loaded.<br />
Like the 26th "Default Applications" would have the same preference as the first "Added Applications"<br />
But this is of low concern here anyway.<br />
It just shows the limitations of the current design based on integer ordering.</p>

<p style="padding: 0; margin: 8px;"><a href="https://phabricator.kde.org/p/ervin/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ervin</a> Do you still feel it needs more comment ?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R309 KService</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D26690</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26690">https://phabricator.kde.org/D26690</a></div></div><br /><div><strong>To: </strong>meven, dfaure, dvratil, ervin, Frameworks<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>