[FFmpeg-devel] [PATCH 4/4] Remove incorrect return statement from avcodec_thread_free()

[Ronald S. Bultje]

Hi,

On Thu, Feb 10, 2011 at 7:22 AM, Mans Rullgard <mans at mansr.com> wrote:
> The function return type is void, so a return statement with an
> expression is forbidden (and pointless).
>
> Signed-off-by: Mans Rullgard <mans at mansr.com>
> ---
> ?libavcodec/utils.c | ? ?2 +-
> ?1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index 03018b8..c4bba37 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -1298,7 +1298,7 @@ int avcodec_thread_init(AVCodecContext *s, int thread_count)
> ?void avcodec_thread_free(AVCodecContext *s)
> ?{
> ?#if HAVE_THREADS
> - ? ?return ff_thread_free(s);
> + ? ?ff_thread_free(s);
> ?#endif
> ?}

Hum, yes, ok.

Ronald