[email protected] (12020-06-08):
> For the print_file is void, so I haven't considered to add error
> check, can we just print one error message if av_bprint_is_complete
> checking failed?

Assume the log goes to /dev/null. If there is an error, especially if
there is a risk of data loss or data corruption, an error return is
necessary. If it requires changing the type of a function, so be it.

But you can do better than using BPrint: the existing code writes into a
buffer then writes the buffer to avio, even though we have
avio_printf().

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to