[FFmpeg-devel] [PATCH] lavf/segment: add reset_timestamps option
Stefano Sabatini
stefasab at gmail.com
Sun Dec 9 12:22:43 CET 2012
On date Thursday 2012-11-29 13:46:53 +0100, Stefano Sabatini encoded:
> The new options reset the timestamps at each new segment, so that the
> generated segments will have timestamps starting from 0.
>
> It is meant to address trac ticket #1425.
> ---
> doc/muxers.texi | 7 +++++++
> libavformat/segment.c | 56 +++++++++++++++++++++++++++++++++++++++++++++++--
> 2 files changed, 61 insertions(+), 2 deletions(-)
Simplified and updated, this should avoid potential desynching
problems, since there is only a single time for each stream, which is
used to recompute timestamps for each stream.
A note about the interface: right now we have the boolean options:
individual_header_trailer
write_header_trailer
reset_timestamps
Would it make sense to set the flag in a segment_flags option,
something like:
-segment_flags +reset_timestamps+individual_header_trailer+write_header_trailer
rather than:
-reset_timestamps 1 -individual_header_trailer 1 -write_header_trailer 1
The former has also the advantage that it avoids potential conflicts
if an option with the same name is added to the global format context.
Also I bet we could find better names for:
individual_header_trailer
write_header_trailer
for example:
wrap_individual
wrap_global
I'll push it in a few days if I receive no replies.
--
FFmpeg = Frenzy & Formidable Minimal Purposeless Emblematic Game
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-lavf-segment-add-reset_timestamps-option.patch
Type: text/x-diff
Size: 5005 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121209/da9ba687/attachment.bin>
More information about the ffmpeg-devel
mailing list