James Almer:
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
> libavcodec/codec.h | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/libavcodec/codec.h b/libavcodec/codec.h
> index bcda061da1..c019e7357c 100644
> --- a/libavcodec/codec.h
> +++ b/libavcodec/codec.h
> @@ -281,6 +281,17 @@ typedef struct AVCodec {
> */
> int (*encode2)(struct AVCodecContext *avctx, struct AVPacket *avpkt,
> const struct AVFrame *frame, int *got_packet_ptr);
> + /**
> + * Decode picture or subtitle data.
> + *
> + * @param avctx codec context
For this to make sense you need to add the variable name in patch 2.
> + * @param outdata codec type dependent output struct
Why didn't you set the "[out]"?
> + * @param[out] got_packet_ptr decoder sets to 0 or 1 to indicate that a
Should be got_frame_ptr. (I am unsure whether we should retain that
"_ptr". It seems superfluous.)
> + * non-empty frame or subtitle was returned in
> + * outdata.
> + * @param[in] avpkt AVPacket containing the bitstream to be
> decoded
Maybe use data instead of bitstream. After all, it may also contain
side-data and other information.
> + * @return amount of bytes read from the packet, negative error code on
> failure
> + */
> int (*decode)(struct AVCodecContext *, void *outdata, int *got_frame_ptr,
> struct AVPacket *avpkt);
> int (*close)(struct AVCodecContext *);
>
_______________________________________________
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".
