[Octave-bug-tracker] [bug #59149] [octave forge] (signal) Resampling from 22050 to 48000 Hz introduces strange and unexpected artifacts

[anonymous]

Follow-up Comment #43, bug #59149 (project octave):

Verified and improved to avoid doubles entirely. A diff to patch the file
src/upfirdn.cc in the signal package is:


46d45
<   const double r = p/(static_cast<double> (q));
58c57
<           const octave_idx_type n = floor (m/r);
---
>           const octave_idx_type n = (m * q / p);


This version gives correct values out of the box without special compiler
flags.

Please verify and push if good.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?59149>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/