[FFmpeg-devel] [PATCH] avutil/dict: Print warning in case av_dict_set() fails to allocate memory

Nicolas George george at nsup.org
Sun Jun 28 17:47:51 CEST 2015

Le decadi 10 messidor, an CCXXIII, Michael Niedermayer a écrit :
> This avoids the need to print such warning in all callers

I believe this is a bad idea: the warning will be duplicated when callers
already include them.

There are low-level APIs that return an error and high-level APIs that print
error messages. Changing the status of an API is not a good idea. Creating a
new API is the way to go.


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

More information about the ffmpeg-devel mailing list