Ignoring the cover on some tracks

[Lukáš Lalinský]

2010/4/1 Thomas Post <post at equinux.com>:
> I attached such a song to this mail.

Thanks, the problem is now fixed in SVN --
http://websvn.kde.org/?view=revision&revision=1110209

Lukas


> On 31.03.2010, at 21:23, Lukáš Lalinský wrote:
>
> On Mon, Mar 29, 2010 at 4:20 PM, Thomas Post <post at equinux.com> wrote:
>
> Hi,
>
> I have some (maybe corrupt) mp4 tracks that have an cover. The cover part
>
> contains a 'data' atom and afterwards a 'name' atom.
>
> Out of interest, could you send me an extract of the file? Or just a
> hexdump of the relevant part?
>
> So the return in the if(name != "data")... should be a break right? That
>
> would ignore the unexpected atom but still adds the maybe previously correct
>
> parsed 'data' atom to d->items. Because (at least) iTunes can read the
>
> covers from these tracks.
>
> That seems reasonable. Continuing if we found an atom different from
> "data" could be dangerous, but stopping and returning what we have so
> far seems fine.
>
> Lukas
>
> Thomas Post