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

[Nicolas George]

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.

> There are at least two such functions already in the code base.

We should not have accepted the second one like that. It does not
constitute a precedent.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210225/a89dc4ec/attachment.sig>