Re: [FFmpeg-devel] [PATCH 1/2] fftools/ffprobe: Add missing dispositions

2021-03-31 Thread Tobias Rapp
On 30.03.2021 19:51, Marton Balint wrote: [...] I wonder if a fate test testing the XML output of ffprobe can be extended to do a validation of the XML via the XSD so this does not gets forgatten... I agree that would be great. But I guess we cannot depend on a tool like xmllint being avai

Re: [FFmpeg-devel] [PATCH 1/2] fftools/ffprobe: Add missing dispositions

2021-03-30 Thread Andreas Rheinhardt
Marton Balint: > > > On Tue, 30 Mar 2021, Andreas Rheinhardt wrote: > >> Andreas Rheinhardt: >>> Signed-off-by: Andreas Rheinhardt >>> --- > >>> diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c >>> index 11e3cbd6c2..38462e1ff3 100644 >>> --- a/fftools/ffprobe.c >>> +++ b/fftools/ffprobe.c >>

Re: [FFmpeg-devel] [PATCH 1/2] fftools/ffprobe: Add missing dispositions

2021-03-30 Thread Marton Balint
On Tue, 30 Mar 2021, Andreas Rheinhardt wrote: Andreas Rheinhardt: Signed-off-by: Andreas Rheinhardt --- diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index 11e3cbd6c2..38462e1ff3 100644 --- a/fftools/ffprobe.c +++ b/fftools/ffprobe.c @@ -2770,6 +2770,11 @@ static int show_stream(Wri

Re: [FFmpeg-devel] [PATCH 1/2] fftools/ffprobe: Add missing dispositions

2021-03-30 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Signed-off-by: Andreas Rheinhardt > --- > fftools/ffprobe.c | 5 + > .../ref/fate/concat-demuxer-extended-lavf-mxf | 2 +- > .../fate/concat-demuxer-extended-lavf-mxf_d10 | 2 +- > .../ref/fate/concat-demuxer-simple1-lavf-mxf | 4 ++-- >

[FFmpeg-devel] [PATCH 1/2] fftools/ffprobe: Add missing dispositions

2021-03-29 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- fftools/ffprobe.c | 5 + .../ref/fate/concat-demuxer-extended-lavf-mxf | 2 +- .../fate/concat-demuxer-extended-lavf-mxf_d10 | 2 +- .../ref/fate/concat-demuxer-simple1-lavf-mxf | 4 ++-- .../fate/concat-demuxer-simple1-la