Skip to content
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

Add `try_reserve` and friends #214

Merged
merged 4 commits into from Apr 23, 2020
Merged

Add `try_reserve` and friends #214

merged 4 commits into from Apr 23, 2020

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Apr 22, 2020

This more cleanly replaces this hack adding fallible allocation from outside the crate:

https://github.com/servo/servo/blob/faa9dccfe8/components/fallible/lib.rs#L101-L163

@emilio
emilio approved these changes Apr 22, 2020
lib.rs Outdated Show resolved Hide resolved
lib.rs Outdated Show resolved Hide resolved
@SimonSapin SimonSapin force-pushed the try_reserve branch from 4ca5036 to 6850790 Apr 22, 2020
@SimonSapin SimonSapin force-pushed the try_reserve branch from ced53e3 to bdfc429 Apr 22, 2020
@SimonSapin
Copy link
Member Author

SimonSapin commented Apr 23, 2020

@bors-servo r=nox

@bors-servo
Copy link
Contributor

bors-servo commented Apr 23, 2020

📌 Commit 3ec778f has been approved by nox

@bors-servo
Copy link
Contributor

bors-servo commented Apr 23, 2020

Testing commit 3ec778f with merge 78c2528...

@bors-servo
Copy link
Contributor

bors-servo commented Apr 23, 2020

☀️ Test successful - checks-travis
Approved by: nox
Pushing 78c2528 to master...

@bors-servo bors-servo merged commit 78c2528 into master Apr 23, 2020
3 checks passed
3 checks passed
Travis CI - Branch Build Passed
Details
Travis CI - Pull Request Build Passed
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the try_reserve branch Apr 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.