[FFmpeg-devel] [PATCH] filters/metadata: add CSV output support

Werner Robitza werner.robitza at gmail.com
Thu Feb 25 13:45:10 EET 2021


On Thu, Feb 25, 2021 at 11:26 AM Nicolas George <george at nsup.org> wrote:
> Werner Robitza (12021-02-24):
> > I didn't really duplicate anything; this was mostly from scratch. The
> > case could be made that a function for escaping CSV could be shared.
>
> The variable names seemed quite similar. Anyway, duplicating a feature
> is just as bad as duplicating code.

Would you accept this patch if this particular function was extracted
to an API and re-used by ffprobe? (And potentially segment.c …)


More information about the ffmpeg-devel mailing list