Skip to content

Commit

Permalink
Auto merge of #124603 - Zalathar:mcdc-mappings, r=Nadrieril
Browse files Browse the repository at this point in the history
coverage: Split out MC/DC mappings from `BcbMappingKind`

These variants were added to `BcbMappingKind` as part of the [MC/DC coverage](https://en.wikipedia.org/wiki/Modified_Condition/Decision_Coverage) implementation in #123409, because that was the path-of-least-resistance for integrating them into the existing code.

However, they ultimately represent complex concepts that the enum was not intended to handle, leading to more complexity in the code that processes them. This PR therefore follows in the footsteps of #124545, and splits the MC/DC mappings out into their own dedicated vectors of structs.

After that, `BcbMappingKind` itself ends up having only one variant (`Code`), so this PR also flattens that enum into its enclosing struct, renamed to `mapping::CodeMapping`.

---

No functional changes.

This will conflict slightly with #124571, but hopefully that should be easy to resolve either way.

`@rustbot` label +A-code-coverage
  • Loading branch information
bors committed May 5, 2024
2 parents b56ef8b + 35aa211 commit 0dc0b2c
Showing 0 changed files with 0 additions and 0 deletions.

0 comments on commit 0dc0b2c

Please sign in to comment.