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

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to