Skip to content

Fix misspelled comments for tests. #16182

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
Aug 10, 2014
Merged

Conversation

jbcrail
Copy link
Contributor

@jbcrail jbcrail commented Aug 2, 2014

No description provided.

@@ -8,7 +8,7 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

// Verify that UnsafeCell is *always* share regardles `T` is share.
// Verify that UnsafeCell is *always* shared regardless if `T` is shared.
Copy link
Member

Choose a reason for hiding this comment

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

I think this one is referring to the trait name, so this should be Share

@steveklabnik
Copy link
Member

You are awesome and this is awesome. r+=me after that one fix

@jbcrail
Copy link
Contributor Author

jbcrail commented Aug 2, 2014

Thanks! I fixed the regression and updated the branch.

@alexcrichton
Copy link
Member

It looks like this may have picked up an extra commit by accident, perhaps a rebase is in order?

@jbcrail
Copy link
Contributor Author

jbcrail commented Aug 3, 2014

One of the build tests failed, but I don't see where my commit contributed. Any suggestions?

@bors bors closed this Aug 10, 2014
@bors bors merged commit 2016742 into rust-lang:master Aug 10, 2014
@jbcrail jbcrail deleted the fix-test-comments branch August 11, 2014 04:17
bors added a commit to rust-lang-ci/rust that referenced this pull request Jan 3, 2024
…Veykril

internal: Update world symbols request definiton, prefer focus range for macros

Prior to this, the symbol search would always jump to the defining macro call, not it jumps to the name in the macro call input if possible. This is a large improvement for assoc items in an attribute impl or trait.
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.

5 participants