[FFmpeg-devel] Yamaha SMAF 'CNTI' chunk is wrong.

[Michael Niedermayer]

On Wed, Jun 24, 2009 at 06:39:21PM +0900, ???? wrote:
> Hi, ffmpeg developers,
> 
> For Yamaha SMAF,  Contents Info chunk ('CNTI') includes only following members:
>   - Contents Class
>   - Contents Type
>   - Contents Code Type
>   - Copy Status
>   - Copy Counts
> 
> (see http://smaf-yamaha.com/what/s52.html)
> 
> But, in the current ffmpeg, 'CNTI' includes metadata ("VN:libavcodec,").
> 
> According to SMAF Specifications Optional Data Chunk
> (http://smaf-yamaha.com/what/s52.html),
> I corrected that metadata include Optional Data Chunk ('OPDA').
> 
> Regards,
> 
> uchan

>  mmf.c |    2 ++
>  1 file changed, 2 insertions(+)
> 37fd3e4e1cdc4068d94570e23672a39bce804ca2  ffmpeg_mmf.patch

ok assuming it doesnt break regression tests and has been tested

besides that we should store LIBAVFORMAT_IDENT when CODEC_FLAG_BITEXACT
is not set, not "libavcodec" (patch for that would be welcome if some
volunteer wants)

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20090625/8a5da77f/attachment.pgp>