[FFmpeg-devel] [PATCH 2/2] avcodec/vorbisenc: Apply dynamic frame lengths

Tyler Jones tdjones879 at gmail.com
Thu Jul 13 17:14:27 EEST 2017


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170713/f11b5ac4/attachment.sig>


More information about the ffmpeg-devel mailing list