On 13/04/16 09:18, nablet developer wrote:
> Signed-off-by: nablet developer <s...@nablet.com>
> ---
>  libavcodec/qsv.c          | 35 +---------------------------
>  libavcodec/qsv_internal.h |  5 ----
>  libavcodec/qsvdec.c       |  1 +
>  libavcodec/qsvenc.c       |  1 +
>  libavutil/Makefile        |  1 +
>  libavutil/qsv_internal.c  | 58 
> +++++++++++++++++++++++++++++++++++++++++++++++
>  libavutil/qsv_internal.h  | 27 ++++++++++++++++++++++
>  7 files changed, 89 insertions(+), 39 deletions(-)
>  create mode 100644 libavutil/qsv_internal.c
>  create mode 100644 libavutil/qsv_internal.h
> 
> diff --git a/libavutil/qsv_internal.h b/libavutil/qsv_internal.h
> new file mode 100644
> index 0000000..de00d09
> --- /dev/null
> +++ b/libavutil/qsv_internal.h
> ...
> +/**
> +  * Convert a libmfx error code into a ffmpeg error code.
> +  */
> +int ff_qsv_error(int mfx_err);

This fails for non-static builds because of the namespace prefix (try building
the shared libraries).

Does this function really need to be available everywhere?  I think you should
wait until you have other patches which actually require it to so that this
change can be assessed properly.  In isolation, it is not useful.

Also, you should have a look at your mail setup.  This arrived today dated six
weeks ago - either it has taken unreasonably long to arrive or something strange
is going on at your sending machine.

Thanks,

- Mark

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

Reply via email to