[FFmpeg-devel] [PATCH] asf chapter support

[Michael Niedermayer]

On Mon, Mar 08, 2010 at 12:12:39PM +0100, Vladimir Pantelic wrote:
> Michael Niedermayer wrote:
>> On Fri, Feb 26, 2010 at 09:56:12AM +0100, Vladimir Pantelic wrote:
>>>  ASF supports "markers" which are a name and a time stamp,
>>>  attached patch uses them to crate lavf chapters.
>>>
>>>  As the markers only provide a start time, I use the start
>>>  time of the next marker or the total duration of the file
>>>  as the end time for the chapter.
>>
>> see compute_chapters_end()
>
> thx, that makes a much shorter patch.
>
>> [...]
>>>  +//av_log(NULL, AV_LOG_ERROR, "add:  start %12lld  end %12lld name 
>>> %s\n", last_pres_time, pres_time, name);
>>
>> ehm
>
> removed, together with some unused variables that were write only.
>
> v2 patch attached.
>

>  asf.c    |    4 ++++
>  asf.h    |    1 +
>  asfdec.c |   27 +++++++++++++++++++++++++++
>  3 files changed, 32 insertions(+)
> 1834f66decb3a2ff318b6131ecea1eac31c3a65f  asf_chapters_v2.patch

ok if tested

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

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- 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/20100308/70b76f82/attachment.pgp>