[FFmpeg-devel] [PATCH] ffprobe: add decode_error_flags

[Tobias Rapp]

On 27.05.2021 20:09, Marton Balint wrote:
> [...]
> 
> An alternative approach is to print the meaning of the actually used
> flags:
> 
> <frame ...>
>   <decode_error_flags invalid_bitstream="0" missing_reference="0" 
> concealment_active="0" decode_slices="0" />
> </frame>
> 
> This is the most readable, but maybe too verbose for the default use?

This would match how pixfmt flags are currently printed by ffprobe. I 
agree that this is readable and avoids duplicating the flag constants on 
the parsing side. But as it would be printed for each frame and takes up 
multiple lines for some output formats I think its quite verbose.

The current output of AVPacket flags takes another approach by printing 
a single-letter character for each flag bit, like 'K' for keyframe and 
'D' for discard. That might be a good compromise, in my opinion.

Regards,
Tobias