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

[Alexander Strasser]

Hi

Stefano Sabatini wrote:
> On date Sunday 2011-09-25 16:26:55 +0200, Alexander Strasser encoded:
> > Stefano Sabatini wrote:
[...]
> > > I added escaping (which may be used e.g. for CSV output), and change
> > 
> >   I think for CSV output it would be better to adhere to RFC 4180.
> > Probably with the exception of "Each line should contain the same
> > number of fields throughout the file.", but that is "should" anyway.
> 
> MS-escaping (as described in RFC4180) is mal-designed and thus I'm not
> eager to make of this the default escaping algo, I suppose the best
> option is to make the escaping algorithm configurable, which requires
> some major redesign.

  If it should be default or not I cannot judge, but it would make sense
because it is what people usually expect when talking about CSV (if that
can at all be said about such a adhoc/no-real-spec file format).

  But as you say, being able to choose the escaping algo would probably
be nice to have.

> I prefer to do this *after* committing this patch though.

  Of course; this comment was not meant to apply to this commit.

  The other comment about escaping key values was though.


  Alexander