[libav-devel] [PATCH 3/7] rtpdec_h264: Cosmetic cleanup

Diego Biurrun diego at biurrun.de
Sat May 5 15:30:54 CEST 2012


On Sat, May 05, 2012 at 02:33:42PM +0300, Martin Storsjö wrote:
> 
> --- a/libavformat/rtpdec_h264.c
> +++ b/libavformat/rtpdec_h264.c
> @@ -98,12 +98,12 @@ static int sdp_parse_fmtp_config_h264(AVStream * stream,
> -    } else  if (!strcmp(attr, "sprop-parameter-sets")) {
> +    } else if (!strcmp(attr, "sprop-parameter-sets")) {
>          uint8_t start_sequence[] = { 0, 0, 0, 1 };
> -        codec->extradata_size= 0;
> -        codec->extradata= NULL;
> +        codec->extradata_size = 0;
> +        codec->extradata = NULL;

You could align here while you're at it.

> @@ -120,49 +120,48 @@ static int sdp_parse_fmtp_config_h264(AVStream * stream,
>  
> -                    codec->extradata= dest;
> -                    codec->extradata_size+= sizeof(start_sequence)+packet_size;
> +                    codec->extradata = dest;
> +                    codec->extradata_size += sizeof(start_sequence) + packet_size;

and here

> -        av_log(codec, AV_LOG_DEBUG, "Extradata set to %p (size: %d)!", codec->extradata, codec->extradata_size);
> +        av_log(codec, AV_LOG_DEBUG, "Extradata set to %p (size: %d)!",
> +                                    codec->extradata, codec->extradata_size);

Indentation is off.

Diego


More information about the libav-devel mailing list