Closed mrannanj closed 3 years ago
Seems fine to me. I will read this through in more detail tomorrow, but if nothing shows up, I think I will just go ahead and merge.
Seems fine to me. I will read this through in more detail tomorrow, but if nothing shows up, I think I will just go ahead and merge.
The ticket said to also look for places where const could be added, but these commits only do the rename.
The ticket said to also look for places where const could be added, but these commits only do the rename.
Yup, that's fine. I like when people make things in small, manageable pieces :)
Also, went through it, nothing jumped at my face immediately.
398