diff mbox

[FFmpeg-devel,3/3] mpegvideo_parser: fix indentation of an if statement

Message ID 20180211143752.7851-4-jeebjp@gmail.com
State New
Headers show

Commit Message

Jan Ekström Feb. 11, 2018, 2:37 p.m. UTC 
From: Masaki Tanaka <maki.rxrz@gmail.com>

---
 libavcodec/mpegvideo_parser.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jan Ekström March 5, 2018, 11:15 p.m. UTC | #1 
On Sun, Feb 11, 2018 at 4:37 PM, Jan Ekström <jeebjp@gmail.com> wrote:
> From: Masaki Tanaka <maki.rxrz@gmail.com>
>
> ---
>  libavcodec/mpegvideo_parser.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/mpegvideo_parser.c b/libavcodec/mpegvideo_parser.c
> index 4f554b684e..ecf7c28e72 100644
> --- a/libavcodec/mpegvideo_parser.c
> +++ b/libavcodec/mpegvideo_parser.c
> @@ -63,7 +63,7 @@ static void mpegvideo_extract_headers(AVCodecParserContext *s,
>                  s->output_picture_number = (buf[0] << 2) | (buf[1] >> 6);
>                  s->pict_type = (buf[1] >> 3) & 7;
>                  if (bytes_left >= 4)
> -                vbv_delay = ((buf[1] & 0x07) << 13) | (buf[2] << 5) | (buf[3]  >> 3);
> +                    vbv_delay = ((buf[1] & 0x07) << 13) | (buf[2] << 5) | (buf[3] >> 3);
>              }
>              break;
>          case SEQ_START_CODE:
> --
> 2.14.3
>

Ping.

Jan
Paul B Mahol March 6, 2018, 8:39 a.m. UTC | #2 
On 3/6/18, Jan Ekstroem <jeebjp@gmail.com> wrote:
> On Sun, Feb 11, 2018 at 4:37 PM, Jan Ekstroem <jeebjp@gmail.com> wrote:
>> From: Masaki Tanaka <maki.rxrz@gmail.com>
>>
>> ---
>>  libavcodec/mpegvideo_parser.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavcodec/mpegvideo_parser.c b/libavcodec/mpegvideo_parser.c
>> index 4f554b684e..ecf7c28e72 100644
>> --- a/libavcodec/mpegvideo_parser.c
>> +++ b/libavcodec/mpegvideo_parser.c
>> @@ -63,7 +63,7 @@ static void
>> mpegvideo_extract_headers(AVCodecParserContext *s,
>>                  s->output_picture_number = (buf[0] << 2) | (buf[1] >> 6);
>>                  s->pict_type = (buf[1] >> 3) & 7;
>>                  if (bytes_left >= 4)
>> -                vbv_delay = ((buf[1] & 0x07) << 13) | (buf[2] << 5) |
>> (buf[3]  >> 3);
>> +                    vbv_delay = ((buf[1] & 0x07) << 13) | (buf[2] << 5) |
>> (buf[3] >> 3);
>>              }
>>              break;
>>          case SEQ_START_CODE:
>> --
>> 2.14.3
>>
>
> Ping.

LGTM
Jan Ekström March 7, 2018, 12:12 a.m. UTC | #3 
On Tue, Mar 6, 2018 at 10:39 AM, Paul B Mahol <onemda@gmail.com> wrote:
> LGTM

Cheers, pushed.

Jan
diff mbox

Patch

diff --git a/libavcodec/mpegvideo_parser.c b/libavcodec/mpegvideo_parser.c
index 4f554b684e..ecf7c28e72 100644
--- a/libavcodec/mpegvideo_parser.c
+++ b/libavcodec/mpegvideo_parser.c
@@ -63,7 +63,7 @@  static void mpegvideo_extract_headers(AVCodecParserContext *s,
                 s->output_picture_number = (buf[0] << 2) | (buf[1] >> 6);
                 s->pict_type = (buf[1] >> 3) & 7;
                 if (bytes_left >= 4)
-                vbv_delay = ((buf[1] & 0x07) << 13) | (buf[2] << 5) | (buf[3]  >> 3);
+                    vbv_delay = ((buf[1] & 0x07) << 13) | (buf[2] << 5) | (buf[3] >> 3);
             }
             break;
         case SEQ_START_CODE: