diff options
| author | Michael Niedermayer <michaelni@gmx.at> | 2013-06-13 12:57:34 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2013-06-13 13:06:22 +0200 |
| commit | 18cb3d355f13df255f0d438c06607b637842e8b5 (patch) | |
| tree | bc92a44b168dd0fc4d69b9a63d189a111ce9401f /libavcodec/4xm.c | |
| parent | e5558391b5aee8e0a99ac10ba9ecd4727439b69a (diff) | |
| parent | 1f0c6075604c271d5627480f1243d22795f9a315 (diff) | |
| download | ffmpeg-18cb3d355f13df255f0d438c06607b637842e8b5.tar.gz | |
Merge commit '1f0c6075604c271d5627480f1243d22795f9a315'
* commit '1f0c6075604c271d5627480f1243d22795f9a315':
4xm: drop pointless assert
Conflicts:
libavcodec/4xm.c
The added condition is impossible to occur and thus is added to the assert.
The assert is not removed from the default case, as the default case cannot
occur. Thus either it should stay as is or the case as a whole be removed.
For extra safety, to ensure this doesnt change the assert is changed to
av_assert0()
Merged-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavcodec/4xm.c')
| -rw-r--r-- | libavcodec/4xm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index dc359c107f..332c10bb24 100644 --- a/libavcodec/4xm.c +++ b/libavcodec/4xm.c @@ -329,7 +329,7 @@ static inline void mcdc(uint16_t *dst, const uint16_t *src, int log2w, } break; default: - av_assert2(0); + av_assert0(0); } } @@ -345,7 +345,7 @@ static int decode_p_block(FourXContext *f, uint16_t *dst, uint16_t *src, uint16_t *end = start + stride * (f->avctx->height - h + 1) - (1 << log2w); int ret; - av_assert0(code >= 0 && code <= 6); + av_assert0(code >= 0 && code <= 6 && log2w >= 0); if (code == 0) { if (bytestream2_get_bytes_left(&f->g) < 1) { |