Hi,

Duplicate branches for if and else is found in libavformat/sol.c, at line 60-62.

static enum AVCodecID sol_codec_id(int magic, int type)
{
    if (magic == 0x0B8D)
    {
        if (type & SOL_DPCM) return AV_CODEC_ID_SOL_DPCM;
        else return AV_CODEC_ID_PCM_U8;
    }
    if (type & SOL_DPCM)
    {
        if (type & SOL_16BIT) return AV_CODEC_ID_SOL_DPCM;
        else if (magic == 0x0C8D) return AV_CODEC_ID_SOL_DPCM;
        else return AV_CODEC_ID_SOL_DPCM;
    }
    if (type & SOL_16BIT) return AV_CODEC_ID_PCM_S16LE;
    return AV_CODEC_ID_PCM_U8;
}

As you can see, all branches return AV_CODEC_ID_SOL_DPCM. Is it correct?


_______________________________________________
ffmpeg-user mailing list
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-user

Reply via email to