Skip to content

Implement the 4 empty function bodies in specs/tri/crypto/hex.t27 - #4124

Closed
gHashTag wants to merge 1 commit into
masterfrom
queen-3761
Closed

gHashTag wants to merge 1 commit into
masterfrom
queen-3761

Conversation

@gHashTag

Copy link
Copy Markdown
Owner

Closes #3761

Opened from the bee branch queen-3761, which was pushed but never turned into a pull request. Every changed path is inside the boundary the issue declares:

  • specs/tri/crypto/hex.t27

The acceptance criteria are structural and were checked by the swarm, not by a human; the review-side oracle is what decides whether the generated code compiles and passes its tests.

- Implemented lower_case() to return Hex with uppercase=false
- Implemented upper_case() to return Hex with uppercase=true
- Implemented encode(codec: Hex) with proper parameter usage
- Implemented decode(input: []const u8) with proper parameter usage
- Added proper test cases for all functions
- All functions now have meaningful implementations instead of TODO placeholders

Closes #3761
@gHashTag
gHashTag enabled auto-merge September 17, 2026 07:56
This was referenced Sep 17, 2026
@gHashTag

Copy link
Copy Markdown
Owner Author

Superseded by #4289, which takes this branch's version of specs/tri/crypto/hex.t27 onto current master.

This pull request could not merge - mergeStateStatus: DIRTY - and a conflicted branch gets no required check, so it reported as waiting for CI. Rather than rebase it, #4289 carries the file, measured with a t27c built from master: it prints IMPLEMENTED with zero not yet implemented, and over the whole corpus NOPARSE goes 65 -> 61.

The work is yours; the branch is just no longer the way it lands.

@gHashTag gHashTag closed this Sep 20, 2026
auto-merge was automatically disabled September 20, 2026 05:17

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement the 4 empty function bodies in specs/tri/crypto/hex.t27

1 participant