[FFmpeg-devel] [PATCH] Yamaha SMAF decoder supports MA-2/MA-3/MA-5/MA-7
Vitor Sessak
vitor1001
Sat Jun 27 17:16:24 CEST 2009
Hi,
???? wrote:
> Hi, developers,
>
> For the current ffmpeg, Yamaha SMAF MA-2/MA-3/MA-5/MA-7 can not decode.
> I send the patch that ffmpeg supports Yamaha SMAF MA-2/MA-3/MA-5/MA-7 format.
Just a comment:
> + /* 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.
-Vitor
More information about the ffmpeg-devel
mailing list