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

Rollup of 25 pull requests #35321

Closed
wants to merge 64 commits into from
Closed

Conversation

GuillaumeGomez and others added 30 commits July 29, 2016 14:32
 * Use make check-lexer to verify the grammar.
 * Extend grammar/README
 * Add make clean-grammar rule
 * Add target check-build-lexer-verifier to make tidy, so it will build the verifier with every build and catch future errors
Beginners may try to adapt the tutorial to develop their own code.
When using different dependencies, they may use the wildcard for
versioning.  Since they are new to the language, they will not know
that the wildcard asterisk is a string, not a token.  Make the correct
format more explicit, to remove one potential source of frustration.
This is a partial fix for issue rust-lang#34808.

Most of the wording was copied verbatim from the RFC. Main difference is that I moved the actual template to the top of the section.

It also makes the error explanations the longest section in the guidelines doc for now.
This adds a question and answer to the Q&A section of the Copy
docs. Specifically, it asks the question I asked while reading
the docs, and gives its answer.
Was surprised to learn that heaps were used in this way, then realized upon reading [the linked paper](http://www.cs.cmu.edu/afs/cs/academic/class/15213-f98/doc/dsa.pdf) that it's a totally different type of heap—an important distinction.
There is no longer a `Repr` trait, so mentioning a missing impl of it
was potentially confusing.
Add Derive not possible question to Copy

This adds a question and answer to the Q&A section of the Copy
docs. Specifically, it asks the question I asked while reading
the docs, and gives its answer.

cc @steveklabnik
…eveklabnik

Add doc examples for FileType struct

Part of rust-lang#29356.

r? @steveklabnik
…klabnik

Provide more explicit example of wildcard version in guessing game doc.

Beginners may try to adapt the tutorial to develop their own code.
When using different dependencies, they may use the wildcard for
versioning.  Since they are new to the language, they will not know
that the wildcard asterisk is a string, not a token.  Make the correct
format more explicit, to remove one potential source of frustration.
A couple `std::net` doc improvements.

None
Add doc example for `std::ffi::NulError::nul_position`.

None
Update the-stack-and-the-heap.md

Was surprised to learn that heaps were used in this way, then realized upon reading [the linked paper](http://www.cs.cmu.edu/afs/cs/academic/class/15213-f98/doc/dsa.pdf) that it's a totally different type of heap—an important distinction.
…-format, r=jonathandturner

E0109 Update error format

Fixes rust-lang#35247 as part of rust-lang#35233.

r? @jonathandturner
Clean up `std::raw` docs

There is no longer a `Repr` trait, so mentioning a missing impl of it was potentially confusing.

r? @steveklabnik
…t, r=jonathandturner

Updated E0071 to new format.

Bonus: the span underlines only the name of the thing that's not a struct rather than the whole expression.

Part of rust-lang#35233.
Fixes rust-lang#35220.

r? @jonathandturner
… r=jonathandturner

E0060 e0061 format update

This fixes rust-lang#35215 and fixes rust-lang#35216 as part of rust-lang#35233

A separate issue will be opened to track the bonus portion of the tickets as @jaredwy will be handling that part.

?r @jonathandturner
…mat, r=jonathandturner

Update error message for E0172

Fixes rust-lang#35272 as part of rust-lang#35233.

r? @jonathandturner
…urner

Update E0124 to the new error format

Part of rust-lang#35233. This resolves rust-lang#35255.

r? @jonathandturner
@steveklabnik
Copy link
Member Author

@bors: r+ p=1

@bors
Copy link
Contributor

bors commented Aug 4, 2016

📌 Commit 60c1910 has been approved by steveklabnik

@rust-highfive
Copy link
Collaborator

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@steveklabnik
Copy link
Member Author

lol @jonathandturner beat me by SECONDS #35320

@steveklabnik steveklabnik deleted the rollup branch August 4, 2016 19:34
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.