add dbaupp's noted overflow to the invalid span list #9

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

Closed

pnkfelix opened this issue Jan 17, 2017 · 2 comments

Owner

pnkfelix commented Jan 17, 2017

Somehow I left out this overflow PR rust-lang/rust#31281 when I made this list here https://github.com/pnkfelix/collab-docs/blob/master/rust/arith-overflow-buglist.md#invalid-span-constructions

Owner Author

pnkfelix commented Jan 17, 2017

Hat tip to @huonw blog post here for this: http://huonw.github.io/blog/2016/04/myths-and-legends-about-integer-overflow-in-rust/

Owner Author

pnkfelix commented Jan 17, 2017

Oh wait, no, the document did have it, a few lines lower, here: https://github.com/pnkfelix/collab-docs/blob/master/rust/arith-overflow-buglist.md#multiline-error-spans

pnkfelix closed this as completed

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