On 19/03/17 07:53, Jun Zhao wrote:
> From 74fa715cdc1e17ab3a59a9e3efa6ef142dd35d1a Mon Sep 17 00:00:00 2001
> From: Jun Zhao <jun.z...@intel.com>
> Date: Sun, 19 Mar 2017 15:44:46 +0800
> Subject: [PATCH] hwcontext: fix comments for av_hwdevice_ctx_alloc()
>
> fix the wrong comments for av_hwdevice_ctx_alloc()
>
> Signed-off-by: Jun Zhao <jun.z...@intel.com>
> ---
> libavutil/hwcontext.h | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/libavutil/hwcontext.h b/libavutil/hwcontext.h
> index 785da09..938c2e4 100644
> --- a/libavutil/hwcontext.h
> +++ b/libavutil/hwcontext.h
> @@ -225,8 +225,7 @@ typedef struct AVHWFramesContext {
> /**
> * Allocate an AVHWDeviceContext for a given pixel format.
> *
> - * @param format a hwaccel pixel format (AV_PIX_FMT_FLAG_HWACCEL must be set
> - * on the corresponding format descriptor)
> + * @param type the type of the hwaccel device.
Right, good catch that this was wrong.
On the replacement, I dislike the use of the word "hwaccel" here because of
confusion with AVHWAccel - this is just the device, and whether it is used for
decoder acceleration or something else isn't defined. (It's slightly
unfortunate that the pixdesc flag is AV_PIX_FMT_FLAG_HWACCEL - something like
HW_OPAQUE would be a much clearer name.)
How about:
"the type of the hardware device to allocate"
?
Thanks,
- Mark
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
