1
73%
main: 73%

Ran 18 Mar 2024 06:37PM UTC

Files 900

Run time 24s

Badge

Embed ▾

Committed 18 Mar 2024 06:18PM UTC coverage: 76.069% (+0.004%) from 76.065%

Job # 8331678711.1

Build Type

push

github

Committed by

web-flow

Commit Message

Add a lossy constructor for tinystr (#4691)

An instance where it is easier for me to write the patch and open a PR
than to start a discussion issue.

I want to be able to write,

```rust
const COMMON_SCRIPT: TinyStr<4> = TinyStr::some_constructor(b"Zyyy");
```

but as far as I can tell, the only way to do this currently would
require the unsafe `from_bytes_unchecked` constructor. I would like to
be able to avoid unsafe, as it introduces additional code review
obligations.


I feel like one possible solution to this would be a lossy constructor,
that handles invalid input by substituting invalid bytes with a valid
replacement.


This PR includes one possible version of this. I'm not at all married to
this particular implementation; it is offered as an illustration.

Coverage Stats

49931 of 65639 relevant lines covered (76.07%)

527080.44 hits per line

unicode-org / icu4x / 8331678711 / 1
73%
main: 73%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8331678711.1

unicode-org / icu4x / 8331678711 / 1 73% main: 73%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8331678711.1

unicode-org / icu4x / 8331678711 / 1
73%
main: 73%

README BADGES
x