[danmar]

No description provided.

[dmcppcheck]

AI review

I found no problems. The PR is a pure refactoring. The {/} scope tracking and the x&=2 default-parameter heuristic moved out of combineOperators() into two static helpers, updateExecutableScope() and isAnonymousReferenceParameter(). The code was moved without changes. The helpers are called only where the old inline code ran, and the continues happen in the same places. The && short-circuit means the backwards scan still runs only for & outside executable scopes, so performance is the same. simplecpp.h is untouched, so the API stays the same, and the code is valid C++11.

Testing

• make -j8 testrunner simplecpp CXXOPTS=-Werror in pr/: built with no warnings.
• ./testrunner: 0 failed (21 skipped, 15 todos).
• python3 run-tests.py: no failures.
• python3 -m pytest integration_test.py: 128 passed.
• ./selfcheck.sh: exit code 0.
• I built base/ and ran both binaries on scratch/a.cpp. It has &=/& = in these places: free function declarations, a declaration with a body, a member declaration in a struct and in a class with a base list, a qualified A::B&= inside a namespace, a qualified call a::b(c&=d), an initializer f(a&=b), an assignment at file scope, and nested blocks in a function. The output of pr and base is identical (diff is empty), so the behaviour has not changed.

---

Automated review of acab82a by Claude. The pull request was built and tested locally.