Skip to content

chore: avoid use of size_t in bbapi to be safe#20916

Merged
ledwards2225 merged 2 commits into
merge-train/barretenbergfrom
lde/size_t
Feb 26, 2026
Merged

chore: avoid use of size_t in bbapi to be safe#20916
ledwards2225 merged 2 commits into
merge-train/barretenbergfrom
lde/size_t

Conversation

@ledwards2225

@ledwards2225 ledwards2225 commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Use uint32_t in bbapi (as was already done in most places) to avoid any platform dependent shenanigans.

@ledwards2225 ledwards2225 marked this pull request as ready for review February 26, 2026 18:12
@ledwards2225 ledwards2225 added the ci-full Run all master checks. label Feb 26, 2026
@AztecBot

Copy link
Copy Markdown
Collaborator

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/07f8574bfe67fa9b�07f8574bfe67fa9b8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_bot.test.ts (160s) (code: 0)

@ledwards2225 ledwards2225 requested a review from ludamad February 26, 2026 19:59
@ledwards2225 ledwards2225 merged commit cd9cbf8 into merge-train/barretenberg Feb 26, 2026
12 checks passed
@ledwards2225 ledwards2225 deleted the lde/size_t branch February 26, 2026 20:16
github-merge-queue Bot pushed a commit that referenced this pull request Feb 27, 2026
BEGIN_COMMIT_OVERRIDE
chore: builders audit response 0 (#20792)
feat: early circuit block deallocation after trace populate (#20776)
fix: Fix script for testing vks haven't changed (#20852)
fix: move debug_incoming_circuit before block deallocation in
Chonk::accumulate (#20876)
refactor: deduplicate debug_incoming_circuit in Chonk::accumulate
(#20870)
fix: denoise causing mysterious error 141 (#20871)
feat: make clang20 preset use zig with glibc 2.35 (#20897)
chore: remove final instances of unneeded +1 in SRS construction
(#20912)
chore: avoid use of size_t in bbapi to be safe (#20916)
chore: Claudebox remedy example (#20922)
END_COMMIT_OVERRIDE
johnathan79717 pushed a commit that referenced this pull request Mar 4, 2026
Use uint32_t in bbapi (as was already done in most places) to avoid any
platform dependent shenanigans.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-full Run all master checks.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants