Tomas Härdin: > static av_always_inline av_const int av_clip_intp2_c(int a, int p) > { > - if (((unsigned)a + (1 << p)) & ~((2 << p) - 1)) > + if (((unsigned)a + (1U << p)) & ~((2U << p) - 1)) > return (a >> 31) ^ ((1 << p) - 1); > else > return a;
This will support p == 30 (but not 31); but the first change is not UB in this range. - Andreas _______________________________________________ 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".