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.

Sign up for GitHub

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

Jump to bottom

Add a test for an invalid, result-producing `if` with no `else` #132

Merged

fitzgen merged 1 commit into rustwasm:master from fitzgen:invalid-if-with-no-else

Sep 26, 2019

Member

fitzgen commented Sep 26, 2019

Wasn't sure that we correctly handled this case, but it turns out we do :)

Worth adding the test anyways (also manually checked that it fails if you uncomment the else).


          Add a test for an invalid, result-producing if with no else

e683578

fitzgen requested a review from alexcrichton

September 26, 2019 21:59

alexcrichton approved these changes

View reviewed changes

fitzgen merged commit 102c133 into rustwasm:master

fitzgen deleted the invalid-if-with-no-else branch

September 26, 2019 22:46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment