ID3 chunks in WAVE/AIFF files

Lukáš Lalinský lalinsky at gmail.com
Mon Nov 1 17:33:21 CET 2010


On Fri, Oct 29, 2010 at 11:34 PM, Stephen F. Booth <me at sbooth.org> wrote:
> On Fri, Oct 29, 2010 at 10:03 AM, Stephen F. Booth <me at sbooth.org> wrote:
>>
>> 2010/10/29 Lukáš Lalinský <lalinsky at gmail.com>
>>>
>>> I'm not sure if this is the best solution. It will read the tag from
>>> 'id3 ', but then write a duplicate 'ID3 '. We should either
>>> consistently use the original chunk ID, or remove existing 'id3 '.
>>
>> You're right- I realized this after I sent the patch and took a look at
>> the save() code.  Having both an 'ID3 ' chunk and an 'id3 ' chunk would be
>> bad.
>> I'm not aware of a standard for the chunk id, so I think the best thing to
>> do in this case is to preserve the original one.  But standardization is
>> also a good argument so I'm on the fence.
>
> I've attached a patch that will keep the existing name of the ID3 chunk.

Thanks. Committed to SVN.

Lukas


More information about the taglib-devel mailing list