<table><tr><td style="">astippich added a comment.
</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/D19109">View Revision</a></tr></table><br /><div><div><p>A few general remarks:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I really do not like that there are two lists of supported mimetypes now which have to be kept in sync</li>
<li class="remarkup-list-item">Do we really need versioning per mimetype? IMHO it is sufficient to have a version number per extractor. From my experience, fixing an extractor usually impacts all its supported mimetypes, and rarily affects only one mimetype. Also, this makes the list hard to maintain, also regarding file types which have multiple mime types, e.g. audio/wav and audio/x-wav</li>
<li class="remarkup-list-item">Do we need an x.y version? I think a single integer is enough or what do you have in mind?</li>
<li class="remarkup-list-item">I prefer to directly construct the qvariantmap in the extractors, and re-use the mimetype list which is already available.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R286 KFileMetaData</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19109">https://phabricator.kde.org/D19109</a></div></div><br /><div><strong>To: </strong>bruns, Baloo, Frameworks, ngraham, astippich, poboiko<br /><strong>Cc: </strong>kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>