-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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 pointing at arg when cause is outside of call #66933
Conversation
r? @eddyb (rust_highfive has picked a reviewer for you, use r? to override) |
b7546cf
to
4cc7ef5
Compare
r? @Centril |
@bors r+ rollup |
📌 Commit 7ec5769254aa1c2aa69eb38dd8b90057dc29ad26 has been approved by |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
7ec5769
to
c808b6a
Compare
src/test/ui/issues/issue-66923-show-error-for-correct-call.stderr
Outdated
Show resolved
Hide resolved
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
d9f8d5b
to
0abb988
Compare
@bors r=Centril |
📌 Commit 0abb988 has been approved by |
Fix pointing at arg when cause is outside of call Closes: rust-lang#66923
Seems failed in #67042 (comment) |
@JohnTitor I think it shall be #66764. |
@lzutao Indeed! So rebase will be needed once it is merged. |
@VirrageS You might want to rebase against latest master and bless the test. |
0abb988
to
e2e4801
Compare
@lzutao done :) |
@bors r=Centril |
📌 Commit e2e4801 has been approved by |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors r- |
here we have 2 errors. One is connected to argument itself
But here we also have similar situation but now we don't want to point to argument but rather to type mismatch which happens where we want to create |
Since I don't have enough knowledge to even try to fix these discrepancies I'm closing this MR. |
The only question is why it worked/compiled before since |
Wouldn't you also bless that test |
The changes are not showing up after I closed the MR but you can check stderr for the blessed test here: VirrageS@e305bf8#diff-83c38304ce39064b906661170076d85e
This is rather confusing than helpful :C |
I do think that the new output in that file is not optimal, but good enough for the change to land, because it isn't misleading, just verbose. |
I can definitelly reopen the new MR (I cannot reopen this for some reason), maybe we can start a new discussion there. I will do that later today. |
Github doesn't let you reopen a pull request that has been closed and then has had it's source branch updated. |
I see :( I've reopened it here: #67354 |
Fix pointing at arg when cause is outside of call Follow up after: rust-lang#66933 Closes: rust-lang#66923 r? @estebank
Fix pointing at arg when cause is outside of call Follow up after: rust-lang#66933 Closes: rust-lang#66923 r? @estebank
Fix pointing at arg when cause is outside of call Follow up after: rust-lang#66933 Closes: rust-lang#66923 r? @estebank
For future reference, you can fix this by force-pushing the previous commit (I believe the syntax is |
Closes: #66923