toml-rs / toml / 15494740973 / 1
81%
main: 81%

DEFAULT BRANCH: main
Ran
Files 107
Run time 6s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 70.023% (+5.3%) from 64.743%
15494740973.1

push

github

web-flow 
refactor(edit): Switch to `toml_parse` (#922)

We don't expect much difference with this change:
- This helps ensure `toml_parse` is as valid as possible
- This paves the way for `toml` to depend on `toml_parse`, rather than
`toml_edit`
- Error messages are changed
- This unblocks us from offering multiple parse errors at once

The quality of error messages is a mixed bag. Some improved while others
got worse. Overall, the worse ones should be the less common ones.

Despite passing the TOML conformance tests, this is a risky change
- It has less reuse between different parts of the parser, so if any
tests assume a feature is covered by another test, it might not be the
case
- Each layer makes assumptions about validation at the other layers
making auditing difficult

## Performance

To reduce extra jitter from allocators being hammered, I ran each
benchmark on its own. Each benchmark was run about 10 times and I pulled
out the fastest result.

```console
$ cargo bench --bench 0-cargo -- toml::document::2-features
```
before: 738us
after: 686us (-7%)

```console
$ cargo bench --bench 0-cargo -- toml::manifest::2-features
```
before: 646us
after: 589us (-9%)

```console
$ cargo bench --bench 0-cargo -- toml_edit::document::2-features
```
before: 527us
after: 477us (-9%)

```console
$ cargo bench --bench 0-cargo -- toml_edit::manifest::2-features
```
before: 650us
after:593us (-9%)

```console
$ cargo bench --bench 1-map -- toml::document::100
```
before: 75us
after: 57us (-24%)

```console
$ cargo bench --bench 1-map -- toml_edit::document::100
```
before: 63us
after:46us (-27%)

```console
$ cargo bench --bench 2-array -- toml::document::100
```
before: 65us
after:46us (-29%)

```console
$ cargo bench --bench 2-array -- toml_edit::document::100
```
before: 61us
after: 41us (-33%)

5487 of 7836 relevant lines covered (70.02%)

8.1 hits per line

Source Files on job 15494740973.1