It looks to me like these arguments are conceptually const. Unless I'm missing something, it seems like declaring them const would make the functions a little more self-documenting.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
It looks to me like these arguments are conceptually const. Unless I'm missing something, it seems like declaring them const would make the functions a little more self-documenting.
I agree. I'll make this change...