astutils.cpp: added more bailouts in `isMutableExpression()` by firewave · Pull Request #6959 · cppcheck-opensource/cppcheck
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given the example above in case of std :: strncmp that will pass any of that to this function and my understanding would be that it makes no sense for std :: in that case. Same for a type like std :: string which I also see.
The case you refer to would be a scoped variable. I wonder if all parts would also be passed as an expression that should be changed.
I have the feeling that namespaces are not handled properly here. I guess it requires some tests to be written.