[FFmpeg-devel] [PATCH 1/3] avformat/avio: add avio_print_n_strings and avio_print

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Aug 11 21:50:59 EEST 2019


On 11.08.2019, at 20:41, Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:

> On 05.08.2019, at 23:34, Marton Balint <cus at passwd.hu> wrote:
> 
>> These functions can be used to print a variable number of strings consecutively
>> to the IO context. Unlike av_bprintf, no temporery buffer is necessary.
> 
> Hm, is there a use-example patch I missed?

Sorry, I see it now, and also the argument about the macro providing type checking.
Ignore me if you feel the code is fine as-is.

> Also is the #define ugliness worth it compared to just requiring NULL termination?
> You can use the sentinel function attribute to have the compiler check that users do not forget it.


More information about the ffmpeg-devel mailing list