On Thu, Jul 13, 2017 at 03:50:13PM +0200, Moritz Barsnick wrote: > On Wed, Jul 12, 2017 at 16:18:20 -0600, Tyler Jones wrote: > > - int window_len = 1 << (venc->log2_blocksize[1] - 1); > > - float n = (float)(1 << venc->log2_blocksize[1]) / 4.0; > > + int prev_size, curr_size, next_size, bound; > > + float scale = 1. / (float) (1 << venc->log2_blocksize[blockflags[1]] - > > 2); > > The "1." is a double, which promotes the calculation to a double > precision operation, which is most likely not intended. Please restrict > it to a float operation by using "1.0f". (I realize the original code > had the same issue.) > > Moritz
You're correct, I'll change that for the next revision. Thank you for catching my mistake. Tyler Jones
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
