> On Oct 20, 2016, at 3:28 AM, Tobias Rapp <t.r...@noa-archive.com> wrote: > > On 19.10.2016 23:15, Michael Niedermayer wrote: >> On Wed, Oct 19, 2016 at 02:35:21PM +0200, Tobias Rapp wrote: >>> From: Rodger Combs <rodger.co...@gmail.com> >>> >>> Reviewed-by: Tobias Rapp <t.r...@noa-archive.com> >>> --- >>> doc/ffprobe.xsd | 1 + >>> ffprobe.c | 13 +++++++++++++ >>> tests/ref/fate/concat-demuxer-extended-lavf-mxf | 2 +- >>> tests/ref/fate/concat-demuxer-extended-lavf-mxf_d10 | 2 +- >>> tests/ref/fate/concat-demuxer-simple1-lavf-mxf | 2 +- >>> tests/ref/fate/concat-demuxer-simple1-lavf-mxf_d10 | 2 +- >>> tests/ref/fate/concat-demuxer-simple2-lavf-ts | 2 +- >>> tests/ref/fate/ffprobe_compact | 4 ++-- >>> tests/ref/fate/ffprobe_csv | 4 ++-- >>> tests/ref/fate/ffprobe_default | 2 ++ >>> tests/ref/fate/ffprobe_flat | 2 ++ >>> tests/ref/fate/ffprobe_ini | 2 ++ >>> 12 files changed, 29 insertions(+), 9 deletions(-) >> >> This too doesnt apply: >> >> Applying: ffprobe: report field order for video streams >> Using index info to reconstruct a base tree... >> error: patch failed: tests/ref/fate/ffprobe_compact:27 >> error: tests/ref/fate/ffprobe_compact: patch does not apply >> error: Did you hand edit your patch? >> It does not apply to blobs recorded in its index. >> > > Could be due to encoding problems or long lines. Have rebased the patch onto > current git HEAD and attached it as a file.
I tested this and the new version applied for me. The output of xml=q=1 validated against the revision. LGTM. Having this data in ffprobe is really helpful. Dave Rice _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
