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

Reinhard Tartler siretart
Thu Feb 10 13:56:55 CET 2011


On Do, Feb 10, 2011 at 13:22:02 (CET), Mans Rullgard 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
>  }

I already wondered, obviously OK.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4




More information about the ffmpeg-devel mailing list