[FFmpeg-devel] [PATCH 4/4] lavf/mov: Add support for edit list parsing.

[Sasi Inguva]

On Sun, Sep 18, 2016 at 6:05 PM, Michael Niedermayer <michael at niedermayer.cc
> wrote:

> On Sun, Sep 18, 2016 at 02:45:52PM -0700, Sasi Inguva wrote:
> > Is it ok to just log an error and return 0 ?
>
> Is a global timescale of 0 allowed ?
> QT does not seem to explicitly say so
>
> can something better be done ?
>
> If you are not sure what to do for the 0 case the best would likely be
> to ask for a sample using avpriv_request_sample()
>
> thx
>
> I created such a file with AtomInspector and played it in QuickTime 10. It
plays the video as "live broadcast". It shows black screen until the first
4s. - the media time of the edit list in the file.
Quicktime 7 says it is an invalid file.

So I think it is not clear how to handle such case. I have updated the code
to use avpriv_request_sample .

Thanks
​
 test_mvhd_timescale_zero_editlist.mov
<https://drive.google.com/a/google.com/file/d/0B47tUwUiOFewcVJsY3d4a2U2Nms/view?usp=drive_web>
​


> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Awnsering whenever a program halts or runs forever is
> On a turing machine, in general impossible (turings halting problem).
> On any real computer, always possible as a real computer has a finite
> number
> of states N, and will either halt in less than N cycles or never halt.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>