-
Notifications
You must be signed in to change notification settings - Fork 123
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
The warnDefaulting option doesn't affect some REPL printing #543
Comments
Yes, These messages are not really warnings, but rather it is Cryptol reporting on what it's doing, much like when it says "Loading...". I can see how they look similar to the other warnings though, even thought the algorithm used is quite different. I don't have a strong opinion if this behavior should be controlled by |
I checked with version 2.5.0, and its behavior matches the current behavior: I guess we could do one of three things:
None of these would be hard to do. Probably the hardest part of 2 would be choosing the name of the new option. |
Does anyone have a strong preference on this? I am leaning towards 1, for simplicity. My thinking is that if you don't care about defaulting in your source code, you probably don't care about the slightly different version at the command line either, and 1 is that path of least surprise, even though it is a little less flexible than 2. |
I'd lean toward 1, as well. It seems like the path of least surprise, as you say. |
I'll go ahead and implement option 1. |
For example:
Maybe the
warnDefaulting
option should control these messages, too? These are, of course, messages that are specific to the REPL. DoeswarnDefaulting
only apply to defaulting that occurs during normal type checking?The text was updated successfully, but these errors were encountered: