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

Stefano Sabatini stefasab at gmail.com
Wed Sep 28 01:57:19 CEST 2011


On date Wednesday 2011-09-28 01:19:06 +0200, Alexander Strasser encoded:
> 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.

Well, actually *this* is my idea, maybe overkill, possibly extensible
to other objects in libav* (e.g. *showinfo filters).

> > 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.
-- 
FFmpeg = Friendly & Fantastic Multimedia Proud Exxagerate Gadget
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-ffprobe-move-writers-to-separate-files-and-use-them.patch
Type: text/x-diff
Size: 29249 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110928/985391c6/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-ffprobe-add-compact-writer.patch
Type: text/x-diff
Size: 5757 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110928/985391c6/attachment-0001.bin>


More information about the ffmpeg-devel mailing list