Message244392
Hi -
During a code review of Modules/audioop.c I noted that weightB is never divided by the GCD while weightA is divided twice. This is contrary to the comment and appears to be a bug:
/* divide weightA and weightB by their greatest common divisor */
d = gcd(weightA, weightB);
weightA /= d;
weightA /= d;
The patch file below (hg diff) fixes this issue. |
|
Date |
User |
Action |
Args |
2015-05-29 17:20:05 | grajagandev | set | recipients:
+ grajagandev |
2015-05-29 17:20:05 | grajagandev | set | messageid: <1432920005.51.0.0928225945144.issue24326@psf.upfronthosting.co.za> |
2015-05-29 17:20:05 | grajagandev | link | issue24326 messages |
2015-05-29 17:20:05 | grajagandev | create | |
|