Skip to content

/ webrender

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

Jump to bottom

Update wrench code to use assert!() instead of if ... { panic!(..) } #727

Closed
glennw opened this issue Jan 17, 2017 · 4 comments
Closed

Update wrench code to use assert!() instead of if ... { panic!(..) } #727

glennw opened this issue Jan 17, 2017 · 4 comments
Labels
difficulty: easy help wanted

Comments

@glennw
Copy link
Member

@glennw glennw commented Jan 17, 2017

No description provided.

@glennw glennw mentioned this issue Jan 17, 2017
Merged
@hgallagher1993
Copy link
Contributor

@hgallagher1993 hgallagher1993 commented Jan 23, 2017 •
edited

Ya I could pick this up. . .does it need a custom error message or just leave it atassert!(nums.len() != 16)?
@mrmiywj
Copy link
Contributor

@mrmiywj mrmiywj commented Mar 29, 2017

I can pick this up.
@mrmiywj mrmiywj mentioned this issue Mar 29, 2017
Merged
@igilham
Copy link

@igilham igilham commented Apr 6, 2017

Looks like this can be closed.
@kvark
Copy link
Member

@kvark kvark commented Apr 6, 2017

@igilham right, thanks for triaging!
Fixed by #1023
@kvark kvark closed this Apr 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
difficulty: easy help wanted
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
@kvark @igilham @glennw @mrmiywj @hgallagher1993
You can’t perform that action at this time.