winnow-rs / winnow

Making parsing a breeze
https://docs.rs/winnow
Other
572 stars 44 forks source link

feat(comb): Allow arrays as input to `alt` #298

Closed epage closed 1 year ago

epage commented 1 year ago

Now that min-const-generics is stabilized, it's possible to (backwards-compatibly) write a version of alt that uses a const array as input instead of being limited to only tuples - this allows us to lift the 21-element limitation on tuples passed to alt.

Upstream: rust-bakery/nom#1556

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 5703371455

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/combinator/branch.rs 10 11 90.91%
<!-- Total: 10 11 90.91% -->
Files with Coverage Reduction New Missed Lines %
src/ascii/mod.rs 2 52.19%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 5650161558: 0.4%
Covered Lines: 1208
Relevant Lines: 2485

💛 - Coveralls
coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 5703328096

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/combinator/branch.rs 10 11 90.91%
<!-- Total: 10 11 90.91% -->
Files with Coverage Reduction New Missed Lines %
src/ascii/mod.rs 2 51.82%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 5650161558: 0.3%
Covered Lines: 1207
Relevant Lines: 2485

💛 - Coveralls
coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 5703328096

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/combinator/branch.rs 10 11 90.91%
<!-- Total: 10 11 90.91% -->
Files with Coverage Reduction New Missed Lines %
src/ascii/mod.rs 2 51.82%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 5650161558: 0.3%
Covered Lines: 1207
Relevant Lines: 2485

💛 - Coveralls