Branch: refs/heads/topic/robin/gh-2107-tidy-toolchain
Home:
https://github.com/zeek/spicy
Commit: be670974020ed7ade7b5b12b3370cc22e1aaff84
https://github.com/zeek/spicy/commit/be670974020ed7ade7b5b12b3370cc22e1aaff84
Author: Robin Sommer <
ro...@corelight.com>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M CMakeLists.txt
Log Message:
-----------
Prettify configure output for sanitizers.
Commit: 54bafb50d3e88c9032968f57c63e2b50e775a496
https://github.com/zeek/spicy/commit/54bafb50d3e88c9032968f57c63e2b50e775a496
Author: Robin Sommer <
ro...@corelight.com>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M CMakeLists.txt
M configure
Log Message:
-----------
Add `--use-clang-tidy` flag to `configure`.
Commit: 59c02e5f98f2979efa3d382c32c25c3ae2ce88fb
https://github.com/zeek/spicy/commit/59c02e5f98f2979efa3d382c32c25c3ae2ce88fb
Author: Robin Sommer <
ro...@corelight.com>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M .clang-tidy
Log Message:
-----------
Enable clang-tidy's naming check.
Not sure why this was excluded: we want it for the toolchain, and it
doesn't seem to hurt for the rest.
Commit: 78cc8fb31a5c375e5dbcdcbc36e7ffdeed059549
https://github.com/zeek/spicy/commit/78cc8fb31a5c375e5dbcdcbc36e7ffdeed059549
Author: Robin Sommer <
ro...@corelight.com>
Date: 2025-09-02 (Tue, 02 Sep 2025)
Changed paths:
M .clang-tidy
M .clang-tidy.toolchain
M hilti/runtime/include/extension-points.h
M hilti/runtime/include/result.h
M hilti/runtime/include/safe-int.h
M hilti/runtime/include/types/bytes.h
M hilti/toolchain/include/ast/builder/node-factory.h
M hilti/toolchain/include/ast/node.h
M hilti/toolchain/include/ast/types/operand-list.h
M hilti/toolchain/include/base/code-formatter.h
M hilti/toolchain/include/base/graph.h
M hilti/toolchain/include/base/util.h
M hilti/toolchain/include/compiler/detail/cfg.h
M hilti/toolchain/include/compiler/jit.h
M hilti/toolchain/src/ast/ast-context.cc
M hilti/toolchain/src/ast/node.cc
M hilti/toolchain/src/ast/operator.cc
M hilti/toolchain/src/ast/operators/bitfield.cc
M hilti/toolchain/src/ast/operators/struct.cc
M hilti/toolchain/src/base/util.cc
M hilti/toolchain/src/compiler/cfg.cc
M hilti/toolchain/src/compiler/codegen/statements.cc
M hilti/toolchain/src/compiler/coercer.cc
M hilti/toolchain/src/compiler/cxx/elements.cc
M hilti/toolchain/src/compiler/cxx/unit.cc
M hilti/toolchain/src/compiler/jit.cc
M hilti/toolchain/src/compiler/optimizer.cc
M hilti/toolchain/src/compiler/validator.cc
M hilti/toolchain/src/
config.cc.in
M hilti/toolchain/tests/util.cc
M spicy/toolchain/include/ast/types/unit-items/block.h
M spicy/toolchain/src/compiler/codegen/parser-builder.cc
M spicy/toolchain/src/compiler/codegen/productions/look-ahead.cc
M spicy/toolchain/src/compiler/codegen/unit-builder.cc
M spicy/toolchain/src/compiler/validator.cc
M spicy/toolchain/src/
config.cc.in
M spicy/toolchain/tests/grammar.cc
Log Message:
-----------
Enforce naming conventions across toolchain using clang-tidy.
This comes with one style clarification: for static, file-local
functions we had an inconsistent mix of prefixing their names with an
underscores vs not. The original intent was to have that prefix, but I
don't see a way to enforce it through clang-tidy, and it's hard to
find all those instances not having the prefix otherwise. So switching
over to not using an underscore prefix.
Compare:
https://github.com/zeek/spicy/compare/be670974020e%5E...78cc8fb31a5c
To unsubscribe from these emails, change your notification settings at
https://github.com/zeek/spicy/settings/notifications