Skip to content

Fix txn_box unused find result#13291

Merged
bneradt merged 1 commit into
apache:masterfrom
bneradt:fix-txn-box-unused-find-result
Jun 18, 2026
Merged

Fix txn_box unused find result#13291
bneradt merged 1 commit into
apache:masterfrom
bneradt:fix-txn-box-unused-find-result

Conversation

@bneradt

@bneradt bneradt commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Clang 22 treats the ignored std::unordered_map::find result in the
txn_box unit perf test as a warning, and the project builds with
warnings promoted to errors. The test therefore fails to compile even
though the lookup is intentionally only being timed.

This explicitly discards the lookup result in the benchmark lambda and
corrects the printed label to describe the lookup operation.

Clang 22 treats the ignored std::unordered_map::find result in the
txn_box unit perf test as a warning, and the project builds with
warnings promoted to errors. The test therefore fails to compile even
though the lookup is intentionally only being timed.

This explicitly discards the lookup result in the benchmark lambda and
corrects the printed label to describe the lookup operation.
Copilot AI review requested due to automatic review settings June 17, 2026 21:10
@bneradt bneradt added this to the 11.0.0 milestone Jun 17, 2026
@bneradt bneradt self-assigned this Jun 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@bneradt bneradt added TxnBox TxnBox plugin Tests labels Jun 17, 2026
@bneradt bneradt merged commit 309d832 into apache:master Jun 18, 2026
15 checks passed
@bneradt bneradt deleted the fix-txn-box-unused-find-result branch June 18, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tests TxnBox TxnBox plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants