[FFmpeg-devel] [PATCH 2/3] ffprobe: report field order for video streams

[Dave Rice]

> On Oct 20, 2016, at 3:28 AM, Tobias Rapp <t.rapp at 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.combs at gmail.com>
>>> 
>>> Reviewed-by: Tobias Rapp <t.rapp at 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