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

Fix typos with repeated words #12727

Merged
merged 1 commit into from
Mar 6, 2014
Merged

Conversation

lifthrasiir
Copy link
Contributor

Cosmetic changes at best, but there are so many such typos that I couldn't ignore them. :) Some occurrences of typos are linked to the generated documentations but no changes should break the builds.

@brson
Copy link
Contributor

brson commented Mar 6, 2014

👍

@huonw
Copy link
Member

huonw commented Mar 6, 2014

@alexcrichton et al.: https://github.com/huonw/spellck, and it even includes a list of known false positives (i.e. differences to my local /usr/share/dict/words).

I guess it could be extended to look for basic grammar issues like doubled words etc. (although there's the problem that that also has false positives, so if we do do it, it needs a list too).

@huonw
Copy link
Member

huonw commented Mar 6, 2014

(Also, this needs a rebase.)

@kud1ing
Copy link

kud1ing commented Mar 6, 2014

One could check the spelling and grammar in a word processor, via pandoc.

@huonw
Copy link
Member

huonw commented Mar 6, 2014

@kud1ing that's a good idea, but doing it quickly (i.e. in an automated fashion, even just to get a list of lines with incorrect words in a crate) seems like it might be tricky.

bors added a commit that referenced this pull request Mar 6, 2014
Cosmetic changes at best, but there are so many such typos that I couldn't ignore them. :) Some occurrences of typos are linked to the generated documentations but no changes should break the builds.
@@ -284,7 +284,7 @@ pub struct PoolConfig {
}

impl PoolConfig {
/// Returns the default configuration, as determined the the environment
/// Returns the default configuration, as determined the environment
Copy link
Contributor

Choose a reason for hiding this comment

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

should be "by the environment".

@bors bors closed this Mar 6, 2014
@bors bors merged commit 1c52c81 into rust-lang:master Mar 6, 2014
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
…eld-name, r=jonas-schievink

fix: Improve suggested names for extracted variables

When extracting a field expression, if RA was unable to resolve the type of the
field, we would previously fall back to using "var_name" as the variable name.

Now, when the `Expr` being extracted matches a `FieldExpr`, we can use the
`NameRef`'s ident token as a fallback option.

fixes rust-lang#10035
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.

None yet

6 participants