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

[Nicolas George]

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.

Regards,

-- 
  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>