Permalink
Browse files

COMMON: Update RDFT and DCT cos/sin table constructor usage

When the constructor for the cos/sin table was changed from
number of bits to number of points all usages thoughout the
code should of been changed, but this was missed in RDFT and DCT.

Fixes Trac#10683.
  • Loading branch information...
dafioram committed Aug 25, 2018
1 parent 82e8bc4 commit 047871f340c8041dce931a39adc3cbb4b6ddd237
Showing with 2 additions and 2 deletions.
  1. +1 −1 common/dct.cpp
  2. +1 −1 common/rdft.cpp
@@ -30,7 +30,7 @@

namespace Common {

DCT::DCT(int bits, TransformType trans) : _bits(bits), _cos(_bits + 2), _trans(trans), _rdft(nullptr) {
DCT::DCT(int bits, TransformType trans) : _bits(bits), _cos(1 << (_bits + 2) ), _trans(trans), _rdft(nullptr) {
int n = 1 << _bits;

_tCos = _cos.getTable();
@@ -28,7 +28,7 @@

namespace Common {

RDFT::RDFT(int bits, TransformType trans) : _bits(bits), _sin(bits), _cos(bits), _fft(nullptr) {
RDFT::RDFT(int bits, TransformType trans) : _bits(bits), _sin(1 << bits), _cos(1 << bits), _fft(nullptr) {
assert((_bits >= 4) && (_bits <= 16));

_inverse = trans == IDFT_C2R || trans == DFT_C2R;

0 comments on commit 047871f

Please sign in to comment.