Building with avcodec version 54 fails.

[cdrummond]

Building with avcodec version 54 fails.

[GoogleCodeExporter]

I've applied a modified version of your patch, needed to a #ifdef's so that it 
still compiles with older avcodec versions. Can you please check that it still 
compiles (r1299) for you - as my avcodec is not as new as yours.

Original comment by [email protected] on 25 Jun 2012 at 6:14

• Changed state: Accepted

[GoogleCodeExporter]

Good thing those #ifdefs I would have done that myself too, but my
C/C++ skills are limited to doing simple function renames and
adding missing headers or copying a snipped of code from another
project when something is not building up ^_^.

Seems you forgot to include the <libavutil/audioconvert.h> header
file which is also needed.

Original comment by [email protected] on 25 Jun 2012 at 7:01

[GoogleCodeExporter]

Oops - added in r1300

Original comment by [email protected] on 25 Jun 2012 at 7:35

[GoogleCodeExporter]

And the ones in the funtion too ^_^
bool FfmpegInput::setChannelMap(int *st) const

Lines 222-234

Original comment by [email protected] on 25 Jun 2012 at 7:38

[GoogleCodeExporter]

What about the 'CH_FRONT_LEFT' ones? Should they be AV_CH_FRONT_LEFT ????

Original comment by [email protected] on 25 Jun 2012 at 7:46

[GoogleCodeExporter]

Yeas like I said the ones in the bool FfmpegInput::setChannelMap(int *st) const
function.

CH_XX_YY -> AV_CH_XX_YY

Original comment by [email protected] on 25 Jun 2012 at 7:50

[GoogleCodeExporter]

Right - your patch never had these, and you said it compiled. Updated in r1301

Original comment by [email protected] on 25 Jun 2012 at 8:00

[GoogleCodeExporter]

It does have them, starting on the line 21......

Original comment by [email protected] on 25 Jun 2012 at 8:01

[GoogleCodeExporter]

Oops :-( you are correct - sorry!!! Anyway, is it all ok now???

Original comment by [email protected] on 25 Jun 2012 at 8:15

[GoogleCodeExporter]

It finally works ^_^

Original comment by [email protected] on 25 Jun 2012 at 8:18

[GoogleCodeExporter]

Original comment by [email protected] on 25 Jun 2012 at 9:28

• Changed state: Fixed