mbox series

[FFmpeg-devel,v3,0/3] Protect against undocumented error codes

Message ID 20240718104757.2809396-1-ffmpeg-devel@pileofstuff.org
Headers show
Series Protect against undocumented error codes | expand

Message

Andrew Sayers July 18, 2024, 10:46 a.m. UTC 
I hadn't noticed av_fourcc2str() before - so long as it's patched to handle "!",
it would be a clear improvement over the earlier patches.

The example in the "!" patch is "BUG!", which is handled in error.c, but shows
that "!" is considered a valid FourCC character.

I think I understand the fate argument now, but this is my first fate patch -
please review it as such!

I still don't see the problem with asking for a bug report.  If this patch is
unacceptable, could you give an example of a code where we would reject a
request to add an error string?