Skip to content

Conversation

@emilio
Copy link
Member

@emilio emilio commented Apr 7, 2017

This change is Reviewable

@emilio
Copy link
Member Author

emilio commented Apr 7, 2017

r? @nox

@emilio
Copy link
Member Author

emilio commented Apr 7, 2017

(Let me know if this should be breaking, but I don't think so)

@nox
Copy link
Contributor

nox commented Apr 7, 2017

It's a breaking change because string_cache::Atom is exposed.

@nox
Copy link
Contributor

nox commented Apr 7, 2017

So it's a breaking change in html5ever and xml5ever too.

@emilio
Copy link
Member Author

emilio commented Apr 7, 2017

Sure, but its API isn't changed, right? Oh, I guess you could have another older string_cache or something like that/

@nox
Copy link
Contributor

nox commented Apr 7, 2017

Yes, using a type from a dependency with a different breaking version than before is a change in API, because Atom isn't the same Atom.

@emilio
Copy link
Member Author

emilio commented Apr 7, 2017

Ammended

@nox
Copy link
Contributor

nox commented Apr 7, 2017

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit fe71bd8 has been approved by nox

@bors-servo
Copy link
Contributor

⌛ Testing commit fe71bd8 with merge 1f5822a...

bors-servo pushed a commit that referenced this pull request Apr 7, 2017
Update string-cache in html5ever-atoms.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/html5ever/264)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: nox
Pushing 1f5822a to master...

@bors-servo bors-servo merged commit fe71bd8 into master Apr 7, 2017
@emilio emilio deleted the string-cache branch April 7, 2017 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants