xtract_smoothness int arguments

[jamiebullock]

xtract_smoothness int arguments

First off, thank you Jaime for all the work you've put into this library over the years, much appreciated!

I'm looking at the here and, while I don't see any specification for an array of integers, the docs do specify the following:

*argv,: pointer to the first element of an array of integers containing the lower bound, upper bound, and pre-scaling factor, > whereby array data in the range lower < n < upper will be pre-scaled by p before processing.

Yet I do not see argv being used in the implementation. Is the documentation out of date (above quote is most recent built from doxygen)

On a side note, I could be wrong but it seems to me that the per-block malloc is unnecessary - you should just be able to cache data[n+1] and iterate over it instead. I would be happy to submit a PR to this affect once there are unit tests in place.

[jamiebullock]

Thanks @reakinator. You are right: the documentation is incorrect. I have also removed the unnecessary call to malloc() as you suggested.

[q-depot]

did you remove the arguments at all? tweaking them was affecting the results before, but I don't know whether it was needed or not.

[jamiebullock]

I think that might be a coincidence. In this commit, I also fixed a bug where *result wasn't being initialised before being used in calculation. It's possible the different results you were getting were just because of that bug.

Although the documentation previously specified arguments for argv, they weren't being used.