[FFmpeg-devel] [PATCH] ffprobe: add csv writer

[Clément Bœsch]

On Wed, Nov 16, 2011 at 06:44:55PM +0100, Stefano Sabatini wrote:
[...]
> > > +static Writer csv_writer = {
> > > +    .name          = "csv",
> > > +    .priv_size     = sizeof(CompactContext),
> > > +
> > > +    .init                  = csv_init,
> > > +    .uninit                = compact_uninit,
> > > +    .print_section_header  = compact_print_section_header,
> > > +    .print_section_footer  = compact_print_section_footer,
> > > +    .print_integer         = compact_print_int,
> > > +    .print_string          = compact_print_str,
> > > +    .show_tags             = compact_show_tags,
> > > +    .flags = WRITER_FLAG_DISPLAY_OPTIONAL_FIELDS
> > > +};
> > > +
> > 
> > Note: I think those writers should be made constant to differentiate them
> > with the context structs (that's not blocking for this patch obviously).
> 
> Yes, patch is welcome, or feel free to commit the change yourself.
> 

Your wish is an order; done, commit and pushed.

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111116/0cb67780/attachment.asc>