Alex Converse <[email protected]> writes:

> On Tue, Sep 6, 2011 at 9:49 AM, Ronald S. Bultje <[email protected]> wrote:
>> Hi,
>>
>> On Tue, Sep 6, 2011 at 2:51 AM, Diego Biurrun <[email protected]> wrote:
>>> ---
>>>  libavcodec/mpeg12.c | 1519 
>>> +++++++++++++++++++++++++--------------------------
>>>  1 files changed, 751 insertions(+), 768 deletions(-)
>>>
>>> diff --git a/libavcodec/mpeg12.c b/libavcodec/mpeg12.c
>>> index 574f69a..2533794 100644
>>> --- a/libavcodec/mpeg12.c
>>> +++ b/libavcodec/mpeg12.c
>>> @@ -1,6 +1,6 @@
>>>  /*
>>>  * MPEG-1/2 decoder
>>> - * Copyright (c) 2000,2001 Fabrice Bellard
>>> + * Copyright (c) 2000, 2001 Fabrice Bellard
>>>  * Copyright (c) 2002-2004 Michael Niedermayer <[email protected]>
>>>  *
>>>  * This file is part of Libav.
>>> @@ -49,28 +49,20 @@
>>>  #define MB_PTYPE_VLC_BITS 6
>>>  #define MB_BTYPE_VLC_BITS 6
>>>
>>> -static inline int mpeg1_decode_block_intra(MpegEncContext *s,
>>> -                              DCTELEM *block,
>>> -                              int n);
>>> -static inline int mpeg1_decode_block_inter(MpegEncContext *s,
>>> -                              DCTELEM *block,
>>> -                              int n);
>>> +static inline int mpeg1_decode_block_intra(MpegEncContext *s, DCTELEM 
>>> *block, int n);
>>> +static inline int mpeg1_decode_block_inter(MpegEncContext *s, DCTELEM 
>>> *block, int n);
>>
>> This doesn't fit on 80 char terminals anymore.
>>
>
> This never fit in 80 chars. Of course you dropped the context that showed 
> this.
>
>>  static inline int mpeg1_fast_decode_block_inter(MpegEncContext *s, DCTELEM 
>> *block, int n);

So break that line after the first comma instead of using it as a
ridiculous excuse for making even more lines hard to read.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to