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

Update atoms #15846

Closed
wants to merge 1 commit into from
Closed

Update atoms #15846

wants to merge 1 commit into from

Conversation

@upsuper
Copy link
Member

upsuper commented Mar 7, 2017

This change is Reviewable

@highfive
Copy link

highfive commented Mar 7, 2017

Heads up! This PR modifies the following files:

  • @bholley: components/style/gecko_string_cache/atom_macro.rs
  • @emilio: components/style/gecko_string_cache/atom_macro.rs
@highfive
Copy link

highfive commented Mar 7, 2017

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
  • These commits modify style code, but no tests are modified. Please consider adding a test!
@upsuper
Copy link
Member Author

upsuper commented Mar 7, 2017

This includes changes from bug 1344964 so it needs to be landed after it.

@upsuper
Copy link
Member Author

upsuper commented Mar 7, 2017

Since I've requested autoland although autoland is closed at the moment, I guess it is fine to just land this.

r? @heycam

@upsuper
Copy link
Member Author

upsuper commented Mar 8, 2017

In favor of #15843.

@upsuper upsuper closed this Mar 8, 2017
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

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