Skip to content

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Mar 23, 2021

fixes: #1724

changelog: Fix false positive with new_ret_no_self when returning Self with different generic arguments

@rust-highfive
Copy link

r? @Manishearth

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Mar 23, 2021
@Jarcho Jarcho force-pushed the new_ret_no_self_fp branch from 1321da5 to 6181a4b Compare March 23, 2021 01:46
Copy link
Member

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

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

Overall seems good! I just think this function could be better named, with a more descriptive comment.

@bors
Copy link
Contributor

bors commented Mar 23, 2021

☔ The latest upstream changes (presumably #6957) made this pull request unmergeable. Please resolve the merge conflicts.

@Jarcho Jarcho force-pushed the new_ret_no_self_fp branch from 4caa00d to 6e88900 Compare March 24, 2021 20:23
@Manishearth
Copy link
Member

@bors r+

thanks!

@bors
Copy link
Contributor

bors commented Mar 24, 2021

📌 Commit 6e88900 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Mar 24, 2021

⌛ Testing commit 6e88900 with merge 4a1825a...

@bors
Copy link
Contributor

bors commented Mar 24, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 4a1825a to master...

@bors bors merged commit 4a1825a into rust-lang:master Mar 24, 2021
@Jarcho Jarcho deleted the new_ret_no_self_fp branch April 6, 2021 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

False positive: methods called new usually return Self
4 participants