[FFmpeg-devel] [PATCH] lavu: header and documentation for AVWriter

Nicolas George george at nsup.org
Wed Sep 7 16:30:09 EEST 2022


Anton Khirnov (12022-09-02):
> As I already said to you in private, I do not think the motivation and
> use cases for this have been sufficiently established.
> 
> You claim this will bring massive advantages all over the place. You
> should support these claims with some actual patches that demonstrate
> these advantages on some real code.

I have not claimed “massive advantages all over the place”, I have
claimed minor advantages all over the places and massive advantages for
future API.

With that correction stated, I have already answered you that your
inability to see the benefits is only proof that your work on FFmpeg is
on areas that rarely need strings. In fact, if I remember correctly, the
last time it happened you used the worst possible implementation and we
had to be multiple persons insisting to get it fixed.

Therefore, I consider your objection to be orders of magnitude less
important than the positive feedback I finally had got from people who
do use strings routinely, who in particular have used BPrint, are
familiar with the benefits it brought but also the limitations I want to
fix.

If there are no more remarks in the next few days I will consider this
approved and start polishing the implementations.

-- 
  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/20220907/4d1b759a/attachment.sig>


More information about the ffmpeg-devel mailing list