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

[Stefano Sabatini]

On date Wednesday 2011-11-16 19:38:12 +0100, Clément Bœsch encoded:
> 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.

Thanks.

-- 
the soft overlord