Oliver Stöneberg
Oliver Stöneberg
Pulled out of #6379. I did not change much in `TestValueFlow` because that would affect almost 1000 lines.
Thanks for your contribution. But this is not we change we would accept as this is clearly for a specific downstream implementation which is is not used in our repo....
No reply in over two weeks - closing. Please re-open if comments are being addressed and there is still something to apply.
I will pull out the global switch for the ValueFlow to a separate PR as it causes a behavior change (albeit only for the CI and my testing). The bailout...
Please point out any additional thresholds I might have missed.
Thanks for contribution. Please update the title of the PR to reflect what it does. And please add a unit test.
I think this looks reasonable and quite unintrusive.
> We could also rename to `.hpp/.hh`. That would be very intrusive and I want to avoid that. I associate `.hpp` with template-heavy (sometimes standalone) headers and it doesn't appear...
Made a draft since a lot of headers are being added.
This is mainly so the issues are reported on start-up instead on each file. It does not improve performance much. The refactoring and improved testing will also make it easier...