[PATCH] D73856: [docu] Improve docu of misc-misplaced-const

2020-02-03 Thread Alexander Lanin via Phabricator via cfe-commits
AlexanderLanin abandoned this revision. AlexanderLanin added a comment. @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. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D73856: [docu] Improve docu of misc-misplaced-const

2020-02-03 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. > East const makes the problem more obvious. With west const people were > telling me the const is on the wrong side. 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

[PATCH] D73856: [docu] Improve docu of misc-misplaced-const

2020-02-02 Thread Alexander Lanin via Phabricator via cfe-commits
AlexanderLanin created this revision. AlexanderLanin added a project: clang-tools-extra. Herald added a project: clang. Herald added a subscriber: cfe-commits. 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