[zeek/spicy] fe652f: Add `--use-clang-tidy` flag to `configure`.

[Robin Sommer]

Branch: refs/heads/topic/robin/gh-2107-tidy-toolchain
Home: https://github.com/zeek/spicy
Commit: fe652f30443a728bc0fcc6bafc0d8e032ea9149b
https://github.com/zeek/spicy/commit/fe652f30443a728bc0fcc6bafc0d8e032ea9149b
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: 27787c25f06d06629451030ecf40b144b1a3e81c
https://github.com/zeek/spicy/commit/27787c25f06d06629451030ecf40b144b1a3e81c
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: 3e8ce2dd9b4cd92db4be8164d595ce9096be9730
https://github.com/zeek/spicy/commit/3e8ce2dd9b4cd92db4be8164d595ce9096be9730
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/aa303b1ba847...3e8ce2dd9b4c

To unsubscribe from these emails, change your notification settings at https://github.com/zeek/spicy/settings/notifications