[Clam-devel] Fixing now manifesting debug asserts

[Hernán Ordiales]

On 8/7/07, Pau Arumi <parumi at iua.upf.edu> wrote:
> Important Note:
> after updating the sandbox you'll need to do scons configure to
> compile in debug.
>
> After the previous fix several debug asserts are manifesting. And
> this commits fixes the ones I've found so far.
> Committed revision 10585.
>
> revision 10585
>   * SinTracking fix (hopefully)
>   ** When mySearch.Find(currentPeakFreq); returns -1 make found=0
>   ** Not sure about what should be done
>
> revision 10584
>   * Array Fix: Constructor with parameter (size) should do
>     Resize(size) after SetSize(size)
>   ** the bug was manifesting in FundFreqDetect in DEBUG
>
> revision 10583
>   * PeakView: Fix: only return a pointer to data when the array size
>     is not 0.
>
>
> The SinTracking bug was the more suspectfull since it seems related
> to PitchShift artifacts. The problem was that
> mySearch.Find(currentPeakFreq) some times returns -1. I'm not sure
> if my fix is correct. Xavier?

pitch-shift isn't asserting now but still with (same) artifacts (i'm
doing the test with elvis voice)... now i'm trying to make it assert
again :-)

just for your knowledge


-- 
Hernán
http://h.ordia.com.ar
GnuPG: 0xEE8A3FE9