[zeek/spicy] be6709: Prettify configure output for sanitizers.

0 views
Skip to first unread message

Robin Sommer

unread,
Sep 2, 2025, 4:53:06 AM (7 days ago) Sep 2
to spicy-...@zeek.org
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
Reply all
Reply to author
Forward
0 new messages