On Mon, 25 Mar 2019 11:17:38 +0100 Michael Niedermayer <[email protected]> wrote:
> On Sun, Mar 24, 2019 at 01:04:51PM +0200, Lauri Kasanen wrote: > > In this function, the exact same clamping happens both in the if and > > unconditionally. > > > > Signed-off-by: Lauri Kasanen <[email protected]> > > --- > > libswscale/output.c | 14 -------------- > > 1 file changed, 14 deletions(-) > > The removed code is the one that should stay, the other should be > removed. > one check for a rarely true condition should be faster than 4 checks Yes, I thought so too, but the commit that added the unconditional code says it fixes a bug... - Lauri _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
