Details
-
Improvement
-
Resolution: Fixed
-
Minor - P4
-
None
-
None
-
Fully Compatible
-
Execution Team 2021-01-25
Description
In SERVER-51240, we removed the const qualifier from a number of function arguments in some of our unit tests - the clang-tidy rule that motivated this change is more of a check on the function declaration rather than the definitions - it is still ok, even meaningful because it communicates how the function argument is used, to retain some of the const qualifiers in the function definitions.
Attachments
Issue Links
- is related to
-
SERVER-51240 Add clang tidy rules to prevent const return by value and (in declarations) const pass by value
-
- Closed
-