Skip to content

perf(parse): Remove extra key allocation#765

Merged
epage merged 1 commit intotoml-rs:mainfrom
epage:features-indexmap
Jul 31, 2024
Merged

perf(parse): Remove extra key allocation#765
epage merged 1 commit intotoml-rs:mainfrom
epage:features-indexmap

Conversation

@epage
Copy link
Member

@epage epage commented Jul 29, 2024

No description provided.

@epage epage force-pushed the features-indexmap branch from 0643304 to 6127ede Compare July 31, 2024 21:18
@epage epage force-pushed the features-indexmap branch from 6127ede to 189697d Compare July 31, 2024 21:20
@epage epage enabled auto-merge July 31, 2024 21:20
@coveralls
Copy link

Pull Request Test Coverage Report for Build 10187827492

Details

  • 115 of 224 (51.34%) changed or added relevant lines in 14 files are covered.
  • 6 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.2%) to 67.843%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/toml_edit/src/index.rs 3 4 75.0%
crates/toml_edit/src/parser/inline_table.rs 5 6 83.33%
crates/toml_edit/src/de/table_enum.rs 4 10 40.0%
crates/toml_edit/src/ser/key.rs 3 19 15.79%
crates/toml_edit/src/table.rs 36 70 51.43%
crates/toml_edit/src/inline_table.rs 29 80 36.25%
Files with Coverage Reduction New Missed Lines %
crates/toml_edit/src/index.rs 1 60.78%
crates/toml_edit/src/item.rs 1 68.1%
crates/toml_edit/src/internal_string.rs 4 29.55%
Totals Coverage Status
Change from base Build 10185912049: 0.2%
Covered Lines: 3711
Relevant Lines: 5470

💛 - Coveralls

@epage epage merged commit ae2dfb4 into toml-rs:main Jul 31, 2024
@epage epage deleted the features-indexmap branch July 31, 2024 21:29
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