-
Notifications
You must be signed in to change notification settings - Fork 28
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
Add more verbose mode for implicits #73
Comments
regarding the second issue, this PR changed the behaviour to print full paths: scala/scala#9767 |
Oooh, nice. So the second issue is gone But a lot of our test will become obsolete. I haven't find a reliable way to remove those:
package prefixes imposed by scala reflection. Any idea how it can be bypassed? I'll add a new pipeline for scala 2.13.7-SNAPSHOT. |
oh, right. I think that might have been one of the reasons why I removed module names in the first place. can't think of a solution on the spot. |
This should be solved, all of our test cases have got rid of the random wrapper. So Unfortunately, I haven't figure out the implementation of |
I just come with the following observation when playing with it for different edge cases:
-Vimplicits-verbose-candidate
In the following example:
this trigger the error report:
Turns out that the default implementation of:
shrug off type signatures of f (it should be
f[Int, String, Char]
). debugging of this error becomes much harder, considering that the following line:Shows only the unreified type signature, the reified version:
Cannot be found anywhere.
-Vimplicits-verbose-position
Since the parameter
keepmodules
is discarded in scala 2.13.6. Package or owner information won't be shown in the message. This means if we have multiple functions likef
org1
with similar signature, it will be difficult to determine which one is the real candidate.This new option should display the line number for each candidate. Making it much harder to cause confusion, e.g. for the above example, the output may become:
The
source-<toolbox>,line-18,offset=218
is generated bycandidate.symbol.pos.showDebug
, obviously this format can be improved.The text was updated successfully, but these errors were encountered: