[FFmpeg-devel] [PATCH] asfenc: fix extended content header

[Michael Niedermayer]

On Sat, Feb 20, 2010 at 01:15:23AM +0100, Anton Khirnov wrote:
> On Fri, Feb 19, 2010 at 11:50:08PM +0100, Michael Niedermayer wrote:
> > 
> > do we loose some metadata that way? if so rejected
> > if no elaborate
> > 
> the file in question contains normal asf tags (UTF16) and then some
> ID3/* tags, where value is an array of [ID3 tag, value ]m both in UTF-8.
> these were never read correctly (before i broke it, ffmpeg tried to read
> them as UTF-16 and produced garbage) and AFAIK aren't standardised in
> any way, so it's IMO safer to skip them than attempt to guess how to
> interpret them.

do you know if this is the only case where this value is used?
iam against skiping cases that where not skiped before without a very
good reason.
And especially "one file putted trash in there" is not a good reason at all
what does the spec say?
do we have other files using this?


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100220/d01e5085/attachment.pgp>