libresample
libresample copied to clipboard
Fix testresample.c output span; add exit code
Prior to this chance, the "Resample with different factors" test only passed for 60 of the 63 factors, with the 3 failing ones being the largest.
- Since only 63 distinct factors were being considered, 100 random samples was overkill.
- To support noticing failure in continuous build systems, it's nice if the test exit()s with nonzero when there are failures.
- The root cause was a formula error when determining which indices in the resampled output ought be compared. Details are explained in a comment.
@minorninth Just a ping to take a look at this; this PR would fix the testing machinery added in https://github.com/minorninth/libresample/pull/8. If you could merge this and tag a 0.1.5 release on the GitHub repository it’d be great for us and other downstreams :)