<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 5, 2013 at 4:34 PM, Paul Harrison <span dir="ltr">&lt;<a href="mailto:mythtv@sky.com" target="_blank">mythtv@sky.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On 06/11/13 00:05, Karl Newman wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
    If the tags are id3 tags then it should find the images in them.<br>
    If they are vorbis (Xiph) tags then we currently don&#39;t support<br>
    reading the Picture blocks from them (taglib does appear to<br>
    support them but nobody has updated our MetaIOFLACVorbis class to<br>
    support them). Of cause patches are welcome :)<br>
<br>
<br>
I guess that&#39;s the issue. I probed the files some more and metaflac finds the tag info but id3 comes up blank. Is taglib used for the id3 tags as well? If so maybe it wouldn&#39;t be too difficult to patch that class since there&#39;s a pattern to follow...<br>


<br>
Karl<br>
<br>
</blockquote>
<br></div>
Yeah both id3 and the flac stuff both use taglib.<br>
<br>
You would need to add the virtual functions from virtual bool supportsEmbeddedImages(void) down to virtual QImage *getAlbumArt(const QString &amp;filename, ImageType type) ( documented here :- <a href="http://code.mythtv.org/cgit/mythtv/tree/mythtv/libs/libmythmetadata/metaio.h#n46" target="_blank">http://code.mythtv.org/cgit/<u></u>mythtv/tree/mythtv/libs/<u></u>libmythmetadata/metaio.h#n46</a> ) into the MetaIOFLACVorbis class.<br>


<br>
You could use the ones in <a href="http://code.mythtv.org/cgit/mythtv/tree/mythtv/libs/libmythmetadata/metaioid3.cpp#n348" target="_blank">http://code.mythtv.org/cgit/<u></u>mythtv/tree/mythtv/libs/<u></u>libmythmetadata/metaioid3.cpp#<u></u>n348</a> as an example.<div class="HOEnZb">

<div class="h5"><br>
<br>
Paul H.<br></div></div></blockquote></div><br></div><div class="gmail_extra">Yep, I found that. It doesn&#39;t look too bad; I&#39;ll see what I can knock together.<br><br>Karl<br></div></div>