East const makes the problem more obvious. With west const people were telling me the const is on the wrong side.
Also it's time to use using instead of typedef.
Differential D73856
[docu] Improve docu of misc-misplaced-const AlexanderLanin on Feb 2 2020, 2:52 PM. Authored by
Details
East const makes the problem more obvious. With west const people were telling me the const is on the wrong side. Also it's time to use using instead of typedef.
Diff Detail Event TimelineComment Actions
I don't see how this makes the problem more obvious and it seems odd to me that we'd mix styles in code and prose. I prefer leaving this as a west const. I have no opinion about typedef vs using in this spot, but given that the check can be used in C as well as C++, typedef is slightly more portable. Comment Actions @aaron.ballman I was ready to complain, but you are right. It would make absolutely no sense to move the const, it's just where it should be. |