NTIA / itm

The Irregular Terrain Model (ITM)
Other
44 stars 25 forks source link

PFL const-correctness #9

Closed JayKickliter closed 1 month ago

JayKickliter commented 12 months ago

PFL can be const since it isn't modified

aromanielloNTIA commented 9 months ago

Thanks for your pull request, @JayKickliter! I just submitted a separate pull request, #12, which implements these changes as well as more const correctness across the codebase. Please feel free to take a look and leave any comments over there.

aromanielloNTIA commented 1 month ago

Closing this since #12 was just merged, which includes this change. Thanks @JayKickliter for the recommendation. If you need this change in the near term, I recommend building ITM from source from the master branch. We do not plan to make a new release in the immediate future to include this change, since some larger refactoring efforts are going on which will take priority.