-
Type: Improvement
-
Resolution: Fixed
-
Priority: Minor - P4
-
Affects Version/s: None
-
Component/s: Internal Code
-
None
-
Fully Compatible
-
Execution Team 2021-01-25
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.
- is related to
-
SERVER-51240 Add clang tidy rules to prevent const return by value and (in declarations) const pass by value
- Closed