[FFmpeg-devel] [PATCH v2] avutil/frame.h: improve documentation for AV_FRAME_FLAG_LOSSLESS

Marton Balint cus at passwd.hu
Sat Jan 11 13:10:11 EET 2025



On Fri, 10 Jan 2025, Leo Izen wrote:

> It should be more clear what this flag is indicating with a more
> verbose comment documenting it.
>
> Signed-off-by: Leo Izen <leo.izen at gmail.com>
> Reviewed-by: Marton Balint <cus at passwd.hu>
> Reviewed-by: Alexander Strasser <eclipse7 at gmx.net>
> Reviewed-by: Marth64 <marth64 at proxyid.net>
> ---
> libavutil/frame.h | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/libavutil/frame.h b/libavutil/frame.h
> index 993fa6b65a..9cd3439e0a 100644
> --- a/libavutil/frame.h
> +++ b/libavutil/frame.h
> @@ -666,7 +666,11 @@ typedef struct AVFrame {
>  */
> #define AV_FRAME_FLAG_TOP_FIELD_FIRST (1 << 4)
> /**
> - * A flag to mark frames which were encoded losslessly from the input.
> + * A decoder can use this flag to mark frames which were originally encoded losslessly.
> + *
> + * For coding bitstream formats which support both lossless and lossy
> + * encoding, it is sometimes possible for a decoder to determine which method
> + * was used when the bitsream was encoded.
>  */
> #define AV_FRAME_FLAG_LOSSLESS        (1 << 5)
> /**
> -- 
> 2.47.1

LGTM, thanks.

Marton


More information about the ffmpeg-devel mailing list