geosolutions-it / jai-ext

Java Advanced Imaging Open Source Replacement Wannabe
Apache License 2.0
90 stars 38 forks source link

The bilinear interpolation code is duplicated between scale and stand-alone billinear interpolator #201

Closed aaime closed 5 years ago

aaime commented 6 years ago

ScaleBilinearOpImage.computeValue and InterpolatorBilinear.computeValue have basically the same signature and the same code (only 9 differences in over 200 lines of code, clear sign of copy/paste/adapt). This means every change to interpolation must be done in two places

For reference, the warp classes are already calling onto the interpolator class instead.