Clarify parameter and argument compatibility#516
Clarify parameter and argument compatibility#516emamoah wants to merge 3 commits intorust-lang:masterfrom
Conversation
Although this is unambiguous after understanding it, I feel it could use a bit more clarity. The first time I read this, I was mostly confused. Many times, without even knowing. Since there was an assignment going on (`*input = val;`), I thought the compatibility was referring to THAT assignment. This proposal makes it clear, especially to new readers, where exactly this compatibility lies.
|
On second thought, this might not be the best clarification, considering that there's no explicit indication of |
|
On third thought (and this demonstrates the subtlety), I do understand that in the assignment, the first |
|
I think I've got a clearer and more accurate suggestion:
It also now uses |
Although this is unambiguous after understanding it, I feel it could use a bit more clarity. The first time I read this, I was mostly confused. Many times, without even knowing. Since there was an assignment going on (
*input = val;), I thought the compatibility was referring to that assignment. This proposal makes it clear, especially to new readers, where exactly this compatibility lies.