On Thu, 16 Jul 2015, Nicolas George wrote:

Le tridi 23 messidor, an CCXXIII, Marton Balint a écrit :
Signed-off-by: Marton Balint <c...@passwd.hu>
---
 doc/demuxers.texi       | 17 +++++++++++++++++
 libavformat/concatdec.c | 39 ++++++++++++++++++++++++++-------------
 2 files changed, 43 insertions(+), 13 deletions(-)


[...]

                            cat->files[fileno - 1].start_time +
                            cat->files[fileno - 1].duration;
+    file->file_start_time = (avf->start_time == AV_NOPTS_VALUE) ? 0 : 
avf->start_time;

+    file->file_inpoint = (file->file_inpoint == AV_NOPTS_VALUE) ? 
file->file_start_time : file->inpoint;

If I read this correctly, file->file_inpoint is always set.

Correct.

[...]

         cat->cur_file->duration = cat->avf->duration;

+        if (cat->cur_file->inpoint != AV_NOPTS_VALUE)

... then this test is always true. Is it on purpose ?

Yes it is, because the check is done on file->inpoint (the inpoint the user provided) not file->file_inpoint which is the pre-calculated inpoint of the file.

 >
+            cat->cur_file->duration -= (cat->cur_file->inpoint - 
cat->cur_file->file_start_time);
+    }


LGTM apart from that.

Ok, thanks. I will send the merge request to Michael.

Regards,
Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to