Skip to content

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Aug 17, 2022

We currently specify non-utf8 str values as UB which is not quite the case,
this PR changes how we describe undefined behavior for valid values of a type
by specifying either a validity invariant or a safety invariant for them.

Closes #150

We currently specify non-utf8 str values as UB which is not quite the case,
this PR changes how we describe undefined behavior for valid values of a type
by specifying either a validity invariant or a safety invariant for them.

Closes #150
Copy link
Contributor

@kirtchev-adacore kirtchev-adacore left a comment

Choose a reason for hiding this comment

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

I like this invariant business!

@kirtchev-adacore
Copy link
Contributor

bors merge

@bors-ferrocene
Copy link
Contributor

Build succeeded:

@bors-ferrocene bors-ferrocene bot merged commit 129edf1 into main Aug 22, 2022
@bors-ferrocene bors-ferrocene bot deleted the validities branch August 22, 2022 13:11
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.

Separate validity and safety invariants in the specification?

2 participants