[FFmpeg-devel] [PATCH] ffprobe: add compact and compactnk writers

Alexander Strasser eclipse7 at gmx.net
Mon Sep 26 02:06:08 CEST 2011


Stefano Sabatini wrote:
> On date Sunday 2011-09-25 16:47:00 +0200, Alexander Strasser encoded:
> >   The problem is that the first field would contains the header
> > ("<bla>: ") or am I missing something?
> > 
> >   In that case, do we have hierarchies like nested sections and
> > therefore nested headers? If not a fix like proposed by Clement
> > which simply drops the ": " and uses the separator would be
> > enough. In awk you could then easily know that the first of the
> > fields is the one containing the section name.
> Oh guys, you managed to convince me, afterall ":" or "|" as header
> separator fits the same and after some tries with awk I convene this
> is a bit simpler.
> >   On the other hand if we have/want hierarchies then some other
> > mechanism needs to be chosen/designed/worked out to store them
> > in flat data formats like CSV, passwd-style etc.
> We don't want them ;-), at least we pretend that we don't need them
> for the compact writers.
> I'll push the patch in a day or so if I read no more comments.

  Sorry to bug you, but I had some comments in my other mail.
Otherwise looks good to me.



More information about the ffmpeg-devel mailing list