Skip to content

Mark C++ exceptions as unsupported in the book #1209

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

Merged
merged 1 commit into from
Jan 5, 2018

Conversation

Ekleog
Copy link
Contributor

@Ekleog Ekleog commented Jan 3, 2018

This should close #1172 . It's my first contribution to the book, so I hope it's not too far from expected usage!

Copy link
Contributor

@emilio emilio left a comment

Choose a reason for hiding this comment

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

LGTM, we need to figure out the travis CI issues that are also hitting #1207.

Thanks Leo! :)

@Ekleog
Copy link
Contributor Author

Ekleog commented Jan 4, 2018

My pleasure, that's not really a lot 😋 but if it can help!

@emilio
Copy link
Contributor

emilio commented Jan 4, 2018

@bors-servo r+

@bors-servo
Copy link

📌 Commit 3f35a93 has been approved by emilio

@bors-servo
Copy link

⌛ Testing commit 3f35a93 with merge ae3b536...

bors-servo pushed a commit that referenced this pull request Jan 4, 2018
Mark C++ exceptions as unsupported in the book

This should close #1172 . It's my first contribution to the book, so I hope it's not too far from expected usage!
@bors-servo
Copy link

💔 Test failed - status-travis

@emilio
Copy link
Contributor

emilio commented Jan 5, 2018

@bors-servo retry

@bors-servo
Copy link

⌛ Testing commit 3f35a93 with merge ccaf192...

bors-servo pushed a commit that referenced this pull request Jan 5, 2018
Mark C++ exceptions as unsupported in the book

This should close #1172 . It's my first contribution to the book, so I hope it's not too far from expected usage!
@bors-servo
Copy link

💔 Test failed - status-travis

@emilio emilio merged commit af78302 into rust-lang:master Jan 5, 2018
@emilio
Copy link
Contributor

emilio commented Jan 5, 2018

Srsly...

@emilio
Copy link
Contributor

emilio commented Jan 5, 2018

Thanks a lot! :)

@Ekleog
Copy link
Contributor Author

Ekleog commented Jan 6, 2018

Heh, sometimes CI isn't all that nice :þ My pleasure :)

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

Successfully merging this pull request may close these issues.

Document (lack of) support for C++ exceptions
4 participants