Improve suggestion for "missing function argument" on multiline call #144966
Conversation
|
|
rustbot
added
S-waiting-on-review
scrabsha
force-pushed
the
push-rozroqqmurvu
branch
from
033e147 to
1eb4aef
Compare
| LL | function_with_lots_of_arguments( | ||
| LL | variable_name, | ||
| LL ~ /* char */, | ||
| LL ~ variable_name, |
There was a problem hiding this comment.
I don't understand why this line is labeled as ~, as it is (well, should be) identical to the initial source. I tried to tweak the code a bit, but found nothing :/
There was a problem hiding this comment.
Because the suggestion I think is being constructed as "/* char */\n " not "\n /* char */", so it's modifying an existing line (where the next arg already is) by adding the new arg suggestion in, then adding the new line with an existing suggestion which also being considered a modification.
You could modify this by appending the line to the previous arg, I think but it would require restructuring your logic.
scrabsha
changed the title
| LL - p3, p4, p5, p6, p7, p8, | ||
| LL - ); | ||
| LL + foo(p1, /* Arc<T2> */, p3, p4, p5, p6, p7, p8); | ||
| LL ~ foo( |
There was a problem hiding this comment.
This case is a bit unfortunate, though I don't know if we'd benefit from tweaking the heuristic to detect "fake" multiline args (i.e. when there's only one line break).
scrabsha
force-pushed
the
push-rozroqqmurvu
branch
from
1eb4aef to
1e271d6
Compare
rustchas a very neat suggestion when the argument count does not match, with a nice placeholder that shows where an argument may be missing. Unfortunately the suggestion is always single-line, even when the function call spans across multiple lines. With this PR,rustctries to guess if the function call is multiline or not, and emits a multiline suggestion when required.r? @jdonszelmann