cppcheck icon indicating copy to clipboard operation
cppcheck copied to clipboard

limit usage of `internal` in selfchecks to files which are being matchcompiled

Open firewave opened this issue 1 year ago • 1 comments

firewave avatar Jun 01 '24 14:06 firewave

There is actually a check which is not dependent on the matchcompiler. checkRedundantNextPrevious() will detect redundant patterns when working on tokens. But I think it should be fine since such code is not being used outside of lib and I think there is no need to apply it to the test code.

firewave avatar Jun 01 '24 15:06 firewave

Splitting cli from lib in the scanning has the unfortunately side effect that the selfcheck is now busy with cli/cmdlineparser.cpp which is the longest file and will no longer process the lib files on the other process. That causes the selfcheck to be slower.

firewave avatar Jul 29 '24 20:07 firewave