[FFmpeg-cvslog] avformat/mux: set avoid_negative_ts based on flags if not overridden by user or muxer

[Nicolas George]

Le septidi 7 floréal, an CCXXI, Michael Niedermayer a écrit :
> ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Fri Apr 26 11:52:51 2013 +0200| [eba33396f007dc373678adea6c1a4b1a50373263] | committer: Michael Niedermayer
> 
> avformat/mux: set avoid_negative_ts based on flags if not overridden by user or muxer
> 
> This changes the default to avoid negative timestamps.

I suggest we consider reverting that: the files produced by the actual
avoid_negative_ts code, which seem to come from the fork, look bogus.

For example:

./ffmpeg_g -lavfi testsrc=d=1 -c libx264 -preset veryfast -crf 20 \
  -avoid_negative_ts 1 -y /tmp/testout.mkv

produces this:

packet|stream_index=0|pts_time=0.080000|dts_time=N/A
packet|stream_index=0|pts_time=0.240000|dts_time=N/A
packet|stream_index=0|pts_time=0.160000|dts_time=0.080000
packet|stream_index=0|pts_time=0.120000|dts_time=0.120000
packet|stream_index=0|pts_time=0.200000|dts_time=0.160000
packet|stream_index=0|pts_time=0.400000|dts_time=0.200000

and this:

format|start_time=0.000000|duration=1.080000

The start_time and duration seems clearly wrong.

It can be compared to the result of producing raw H.264 and muxing with
mkvmerge:

packet|stream_index=0|pts_time=0.000000|dts_time=N/A
packet|stream_index=0|pts_time=0.160000|dts_time=N/A
packet|stream_index=0|pts_time=0.080000|dts_time=0.000000
packet|stream_index=0|pts_time=0.040000|dts_time=0.040000
packet|stream_index=0|pts_time=0.120000|dts_time=0.080000
packet|stream_index=0|pts_time=0.320000|dts_time=0.120000
...
format|start_time=0.000000|duration=1.000000

Using "-avoid_negative_ts 0" produces the same result as mkvmerge.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20131016/8ba03828/attachment.asc>