[FFmpeg-devel] [PATCH] lavf/http: Fix incorrectly placed parenthesis.

wm4 nfxjfg at googlemail.com
Thu Sep 3 16:19:06 CEST 2015


On Thu,  3 Sep 2015 15:55:10 +0200
Stephan Holljes <klaxa1337 at googlemail.com> wrote:

> Signed-off-by: Stephan Holljes <klaxa1337 at googlemail.com>
> ---
>  libavformat/http.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> The assignment had incorrectly placed parentheses which resulted in ret
> always being > 0.

Stuff like this can go into the commit message too.

> diff --git a/libavformat/http.c b/libavformat/http.c
> index d3c0b7e..c148a29 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -399,7 +399,7 @@ static int http_handshake(URLContext *c)
>      switch (ch->handshake_step) {
>      case LOWER_PROTO:
>          av_log(c, AV_LOG_TRACE, "Lower protocol\n");
> -        if ((ret = ffurl_handshake(cl) > 0))
> +        if ((ret = ffurl_handshake(cl)) > 0)
>              return 2 + ret;
>          if ((ret < 0))
>              return ret;

LGTM


More information about the ffmpeg-devel mailing list