Remove unused jdmath functions?

Chandra-MARX / marx

Chandra X-ray Observatory ray-trace simulator

http://space.mit.edu/cxc/marx/

5 stars 4 forks source link

Remove unused jdmath functions? #74

Open hamogu opened 11 months ago

hamogu commented 11 months ago

jdmath contains some functions that are not used in marx. Is is worthwhile to remove them to reduce maintenance? Below is a list of functions as I find them, but a systematic review may reveal more:

hist.c: None of the histogram functions are used, except in the demo program to demo rebinning

hamogu commented 11 months ago

Interpolation is defined twice, once for floats and once for doubles in finterpo.c and dinterpo.c. Any fixes to one of those files should probably be applied to both. hist.c shows an approach of using the preprocessor to use the same code for two different data types; on the one hand that's less duplication, but it's also harder to read.