On Tue, 6 Aug 2019, Nicolas George wrote:

Marton Balint (12019-08-05):
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.

I do not like this very much: the VA design disables type checking,
which makes it rather fragile, and the benefit is really minor.

This is only a problem if avio_print_n_strings is called directly, but the user should always use avio_print. And avio_print macro is casting every parameter to const char *, so the compiler emits a warning if the user passes an argument which is not const char *.


Maybe add avio_print_string() to print a single string, and call it as
many times as necessary.

That reduces readability slightly, so I'd rather not do it.

Regards,
Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to