Skip to content

Clarify wordings of the unstable_name_collision lint. - #50360

Merged
bors merged 1 commit into
rust-lang:masterfrom
kennytm:fix-50232-clarify-unstable-name-collision
May 3, 2018
Merged

Clarify wordings of the unstable_name_collision lint.#50360
bors merged 1 commit into
rust-lang:masterfrom
kennytm:fix-50232-clarify-unstable-name-collision

Conversation

@kennytm

Copy link
Copy Markdown
Member

Stabilizing an inherent method may cause change in behavior instead of inference error. Updated to use the wording from varkor's comment.

Closes#50232.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @nikomatsakis

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

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 1, 2018
Stabilizing an inherent method may cause change in behavior instead of
inference error. Updated to use the wording from [varkor's comment].
Closesrust-lang#50232.
[varkor's comment]: rust-lang#50232 (comment)
@kennytm
kennytmforce-pushed the fix-50232-clarify-unstable-name-collision branch from 0366d27 to a78028dCompareMay 1, 2018 08:54
@nikomatsakis

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented May 1, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit a78028d has been approved by nikomatsakis

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 1, 2018
@kennytmkennytm mentioned this pull request May 3, 2018
kennytm added a commit to kennytm/rust that referenced this pull request May 3, 2018
…name-collision, r=nikomatsakis
Clarify wordings of the `unstable_name_collision` lint.
Stabilizing an inherent method may cause change in behavior instead of inference error. Updated to use the wording from [varkor's comment].
Closesrust-lang#50232.
[varkor's comment]: rust-lang#50232 (comment)
bors added a commit that referenced this pull request May 3, 2018
Rollup of 12 pull requests
Successful merges:
- #50302 (Add query search order check)
- #50320 (Fix invalid path generation in rustdoc search)
- #50349 (Rename "show type declaration" to "show declaration")
- #50360 (Clarify wordings of the `unstable_name_collision` lint.)
- #50365 (Use two vectors in nearest_common_ancestor.)
- #50393 (Allow unaligned reads in constants)
- #50401 (Revert "Implement FromStr for PathBuf")
- #50406 (Forbid constructing empty identifiers from concat_idents)
- #50407 (Always inline simple BytePos and CharPos methods.)
- #50416 (check if the token is a lifetime before parsing)
- #50417 (Update Cargo)
- #50421 (Fix ICE when using a..=b in a closure.)
Failed merges:
@bors
bors merged commit a78028d into rust-lang:masterMay 3, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@kennytm@rust-highfive@nikomatsakis@bors