Skip to content

Conversation

@tarcieri
Copy link
Member

Depends on RustCrypto/utils#1266

Extracts all of the const fn constructor and predication/selection methods used by crypto-bigint into the ctutils crate, allowing the ConstChoice type to be removed and replaced by ctutils::Choice.

@codecov
Copy link

codecov bot commented Dec 20, 2025

Codecov Report

❌ Patch coverage is 93.70079% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.13%. Comparing base (1114f7c) to head (275cca2).

Files with missing lines Patch % Lines
src/ct.rs 85.71% 16 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1035      +/-   ##
==========================================
- Coverage   79.23%   79.13%   -0.10%     
==========================================
  Files         164      165       +1     
  Lines       17707    17616      -91     
==========================================
- Hits        14030    13941      -89     
+ Misses       3677     3675       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tarcieri tarcieri force-pushed the replace-constchoice-with-ctutils-choice branch 6 times, most recently from fb13ecd to 9869e41 Compare December 20, 2025 20:12
Depends on RustCrypto/utils#1266

Extracts all of the `const fn` constructor and predication/selection
methods used by `crypto-bigint` into the `ctutils` crate, allowing the
`ConstChoice` type to be removed and replaced by `ctutils::Choice`.
@tarcieri tarcieri force-pushed the replace-constchoice-with-ctutils-choice branch from 9869e41 to 275cca2 Compare December 20, 2025 20:25
@tarcieri tarcieri marked this pull request as ready for review December 20, 2025 20:25
@tarcieri tarcieri changed the title [WIP] Replace ConstChoice with ctutils::Choice Replace ConstChoice with ctutils::Choice Dec 20, 2025
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.

2 participants