[FFmpeg-soc] [soc]: r5663 - mms/mmst.c

[Ronald S. Bultje]

Hi,

On Mar 13, 2010, at 8:22 AM, Thilo Borgmann <thilo.borgmann at googlemail.com 
 > wrote:
> Am 13.03.10 14:07, schrieb spyfeng:
>> Author: spyfeng
>> Date: Sat Mar 13 14:07:31 2010
>> New Revision: 5663
>>
>> Log:
>> remove "{" and "}" when return -1 directly.
>
> My latest news is to always use
>
> if () {
> ...
> } else {
> ...
> }
>
> even for one-liners to reduce patch size.
> Using
>
> if () {
> ...
> } else
> ... ;
>
> never was a valid choice and AFAIK it still is none.

It reduces codesize by 1 line and is thus better.

More specifically , I asked for it because this MMS patch is seriously  
massively huge and needs some fat trimming.

Ronald