Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Allow (opt-in) to run
clang-tidy
on Arbor.Limitations currently are:
lmorpho
modcc
tests
(clang-tidy doesn't always seeEXPECT_EQ(a, b)
as a use ofa
andb
).clang-tidy
into the src-dirsclang-analyzer-*
.clang-tidy
but require fixing more issues.std::vector<size_t>(size, val) => {size, val}
Sensible suggestions were implemented and as only two
weird ones remained, I massaged the code to appease the
Linter despite being quite sure of a false positive.
Things left disabled can be toggled on after a while.