On Mon, Jul 27, 2020 at 11:08:09AM +0200, Andreas Rheinhardt wrote: > instead of get_ue_golomb(). The difference between the two is that the > latter also has to take into account the case in which the read code is > more than 9 bits (four preceding zeroes + at most five value bits) long, > leading to more code. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > --- > libavcodec/cavsdec.c | 2 +- > libavcodec/h264_parse.c | 4 ++-- > libavcodec/h264_ps.c | 24 ++++++++++++------------ > libavcodec/hevc_parser.c | 2 +- > 4 files changed, 16 insertions(+), 16 deletions(-) > > diff --git a/libavcodec/cavsdec.c b/libavcodec/cavsdec.c > index aaed807196..9c3825df38 100644 > --- a/libavcodec/cavsdec.c > +++ b/libavcodec/cavsdec.c > @@ -676,7 +676,7 @@ static int decode_mb_i(AVSContext *h, int cbp_code) > } > h->pred_mode_Y[pos] = predpred; > } > - pred_mode_uv = get_ue_golomb(gb); > + pred_mode_uv = get_ue_golomb_31(gb); > if (pred_mode_uv > 6) { > av_log(h->avctx, AV_LOG_ERROR, "illegal intra chroma pred mode\n"); > return AVERROR_INVALIDDATA;
LGTM i dont think the others are speed relevant so they could be changed or left as they are thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Dictatorship naturally arises out of democracy, and the most aggravated form of tyranny and slavery out of the most extreme liberty. -- Plato
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".