[FFmpeg-devel] [PATCH] movenc.c fix (Was: Can av_write_frame() modify pkt.data?)

Baptiste Coudurier baptiste.coudurier
Wed Jul 23 19:12:43 CEST 2008


Hi Luca,

Luca Abeni wrote:
> Hi Baptiste,
> 
> Baptiste Coudurier wrote:
> [...]
>>> I now see how your idea can simplify the code. Unfortunately, I got
>>> sidetracked by some other issues, and I've not been able to test these
>>> changes. I attach a patch, to show how the final result will look like,
>>> but be aware that the patch is only compile-tested.
>>> I am not sure I like the way I changed ff_isom_write_avcc()... I'll have
>>> to think about it a little bit more.
>>>
>>> I'll probably have a tested patch next week.
>>>
>>> [...]
>>>
>> Well at first sight, I'd say first patch not using ByteIOContext is simpler.
> 
> Sorry for the long silence about this issue... I've been busy with other
> stuff. In the next days, I'll hopefully have more time for cleaning up
> the patch and testing it. I'll probably post both the two patches (the one
> using ByteIOContext, and the first one) so that people can decide which is
> better.
> 
> Just a question: which kind of tests should I run to be sure that my patches
> do not break anything? Is "make test" enough, or there is some additional
> test that I should run to test this code?
> 

Well, ensure that created file is playable by every player including
quicktime I'd say.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA




More information about the ffmpeg-devel mailing list