[FFmpeg-devel] [PATCH] ffprobe: generalize writer subsection nesting model
Stefano Sabatini
stefasab at gmail.com
Wed Sep 26 11:11:31 CEST 2012
On date Wednesday 2012-09-26 01:07:18 +0200, Stefano Sabatini encoded:
> On date Tuesday 2012-09-25 21:39:00 +0200, Clément Bœsch encoded:
[...]
> > OK, that's kind of a huge patch. I'll hardly be able to make a deeper
> > review. Feel free to drop any of my comments if you plan to fix them in a
> > later patch, and if passes FATE.
> >
> > Just an extra check before you push, I'm always a bit worried about stuff
> > like a missing trailing comma in the json output in some corner cases,
> > like when you have metadata but empty, or no metadata at all (and so the
> > section is not printed). Also when you mixed try various different
> > combinations of the -show_* options.
> >
> > Anyway, good work so far, thanks.
>
> I'll do more tests, and push when I feel the patch can be considered
> ready.
Pushed.
--
FFmpeg = Fundamentalist and Free Most Portentous Enlightening Gem
More information about the ffmpeg-devel
mailing list