[FFmpeg-devel] [PATCH] Yamaha SMAF decoder supports MA-2/MA-3/MA-5/MA-7

Vitor Sessak vitor1001
Sun Jun 28 11:40:17 CEST 2009


???? wrote:
> Hi,
> 2009/6/28 Vitor Sessak <vitor1001 at gmail.com>:
> 
>>> +        /* Make sure it's followed by an Awa chunk, aka wave data */
>>> +        if ((tag & 0xffffff) != MKTAG('A', 'w', 'a', 0)) {
>>> +            av_log(s, AV_LOG_ERROR, "Unexpected SMAF chunk %08x\n", tag);
>>> +            return -1;
>>> +        }
>>>     }
>>>  -    /* Make sure it's followed by an Awa chunk, aka wave data */
>>> -    if ((tag & 0xffffff) != MKTAG('A', 'w', 'a', 0)) {
>>> -        av_log(s, AV_LOG_ERROR, "Unexpected SMAF chunk %08x\n", tag);
>>> -        return -1;
>>> -    }
>> This block has just been reindented. Please do not mix functional and
>> indentation/cosmetics changes in patches (even if it will result in a
>> temporarily broken indentation) or else they are much harder to review.
> 
> The above block is not cosmetic. Patch applied source is following:

I was referring only to the 5+ and 5- I quoted. Which one of the 
following 5 lines was actually changed beyond indentation? If none, it 
is a cosmetical change.

> +        /* Make sure it's followed by an Awa chunk, aka wave data */
> +        if ((tag & 0xffffff) != MKTAG('A', 'w', 'a', 0)) {
> +            av_log(s, AV_LOG_ERROR, "Unexpected SMAF chunk %08x\n", tag);
> +            return -1;
> +        }
>      }
>  
> -    /* Make sure it's followed by an Awa chunk, aka wave data */
> -    if ((tag & 0xffffff) != MKTAG('A', 'w', 'a', 0)) {
> -        av_log(s, AV_LOG_ERROR, "Unexpected SMAF chunk %08x\n", tag);
> -        return -1;
> -    }

-Vitor



More information about the ffmpeg-devel mailing list