Fixes CID741418
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Slightly simplify readability, since the initial check is unnecessary.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

It does not make sense to set a negative bitrate.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Simplify, increase robustness in case a new sample format is added.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

f174fbac did not introduce DR1 flag it was already available before that.

This reverts commit 8bf44d5c.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Signed-off-by: Paul B Mahol <onemda@gmail.com>

In ISO/IEC 13818-2, bit rate is differently determined according to video type

1) MPEG1 Video
 Bit_rate and vbv_delay are set to 3FFFF and FFFF respectively
 to indicate variable bitrate. Other values are for constant bitrate.
 VBV is only defined for constant bit rate operation.
 Ths STD supersedes the VBV model for vbr.

2) MPEG2 Video
 Even if the bitrate is constant, the value of bit_rate may not be the actual bitrate
 since bit_rate need only be an upper bound to that actual bitrate.
 VBV is only defined for variable bit rate operation.
 Constant bit rate is viewed as a special case of vbr.
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Timestamp handling of decoded frames are much simpler now.

Fixes #1675.
Signed-off-by: Paul B Mahol <onemda@gmail.com>

Useful for detection of subtitles displayed during foreign language
scenes.
Signed-off-by: Diego Biurrun <diego@biurrun.de>

Requested in trac ticket #1857.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Same fix and issue as in a25d912dSigned-off-by: Anton Khirnov <anton@khirnov.net>

A large invalid value could cause undefined behavior when left-shifted
by 8 later in the function.

channels, sample_rate, bit_rate, and block_align can be used directly from
the AVCodecContext

This could occur due to integer overflow when reading the channel count from
the extradata.