Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix interaction between mir_fresh_expanded_value and repr(transparent) structs #1974

Merged
merged 1 commit into from
Nov 11, 2023

Conversation

RyanGlScott
Copy link
Contributor

Previously, if mir_fresh_expanded_value was asked to generate a value whose type was a struct marked repr(transparent), then SAW would produce a value of the underlying type, not the struct type. This fixes the issue by wrapping the underlying value with SetupStruct.

Fixes #1973.

…ent)` structs

Previously, if `mir_fresh_expanded_value` was asked to generate a value whose
type was a struct marked `repr(transparent)`, then SAW would produce a value of
the underlying type, not the struct type. This fixes the issue by wrapping the
underlying value with `SetupStruct`.

Fixes #1973.
Copy link
Contributor

@qsctr qsctr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@RyanGlScott RyanGlScott added the PR: ready to merge Magic flag for pull requests to ask Mergify to merge given an approval and a successful CI run label Nov 11, 2023
@mergify mergify bot merged commit 0202016 into master Nov 11, 2023
39 checks passed
@mergify mergify bot deleted the T1973 branch November 11, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready to merge Magic flag for pull requests to ask Mergify to merge given an approval and a successful CI run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mir_fresh_expanded_value produces value of incorrect type for repr(transparent) structs
2 participants