[FFmpeg-devel] [PATCH] avcodec: list the allocation requirements for intra_matrix and inter_matrix fields

[Paul B Mahol]

LGTM

On 7/15/19, James Almer <jamrial at gmail.com> wrote:
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
> See
> https://git.videolan.org/?p=vlc.git;a=commitdiff;h=d86c4c87aa78130a4fd00294e25df865d0e2b327
>
>  libavcodec/avcodec.h | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index 2528bd89ab..646b8f05fc 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -2057,15 +2057,19 @@ typedef struct AVCodecContext {
>
>      /**
>       * custom intra quantization matrix
> -     * - encoding: Set by user, can be NULL.
> -     * - decoding: Set by libavcodec.
> +     * Must be allocated with the av_malloc() family of functions, and will
> be freed in
> +     * avcodec_free_context().
> +     * - encoding: Set/allocated by user. Freed by libavcodec. Can be NULL.
> +     * - decoding: Set/allocated/freed by libavcodec.
>       */
>      uint16_t *intra_matrix;
>
>      /**
>       * custom inter quantization matrix
> -     * - encoding: Set by user, can be NULL.
> -     * - decoding: Set by libavcodec.
> +     * Must be allocated with the av_malloc() family of functions, and will
> be freed in
> +     * avcodec_free_context().
> +     * - encoding: Set/allocated by user. Freed by libavcodec. Can be NULL.
> +     * - decoding: Set/allocated/freed by libavcodec.
>       */
>      uint16_t *inter_matrix;
>
> --
> 2.22.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".