<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Scott,</div><div><br></div>I can confirm that the patch fixes certain files but breaks other ones. I have a few files I could send you whose album art is displayed once the patch is applied.<div><br><div><div>On 30-Oct-08, at 5:45 PM, Scott Wheeler wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Maxim Bourmistrov wrote:<br><blockquote type="cite">Tested with PNG-image inside APIC-frame.<br></blockquote><blockquote type="cite">Here is a detailed description with some nice screenshots :) -> <a href="http://en.roolz.org/Blog/Entries/2008/10/29_Using_taglib_to_extract_cover_image_from_MP3-files.html">http://en.roolz.org/Blog/Entries/2008/10/29_Using_taglib_to_extract_cover_image_from_MP3-files.html</a><br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">I'm not on this list, so please replay to my mail directly.<br></blockquote><blockquote type="cite"><br></blockquote><br>Hi Maxim --<br><br>This patch looks incorrect. My guess would be that iTunes (if that's <br>what put the images there) is storing the image incorrectly in the <br>frame. Its ID3v2 implementation is rather buggy. I'd assume that <br>either they're not writing the description or its not being properly <br>terminated. I just checked the spec and the APIC frame has been <br>consistent across ID3v2.2 - ID3v2.4, so it shouldn't be an <br>incompatibility there.<br><br>Could you send a sample file that's problematic to me via mail? I'd <br>like to have a glance at it with a hex editor to see what's up.<br><br>Cheers,<br><br>-Scott<br>_______________________________________________<br>taglib-devel mailing list<br><a href="mailto:taglib-devel@kde.org">taglib-devel@kde.org</a><br><a href="https://mail.kde.org/mailman/listinfo/taglib-devel">https://mail.kde.org/mailman/listinfo/taglib-devel</a><br></div></blockquote></div><br></div></body></html>