-
-
Notifications
You must be signed in to change notification settings - Fork 662
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 -w #10612
Add -w #10612
Conversation
Might be good to "reserve" more than 100 per category. Also since you are doing this: error codes are useful too, they make it really easy to uniquely identify an error and document it in a manual. |
I agree. I actually wouldn't mind doing the dumb labor required for that, but somehow I'm always very reluctant to assign unique numbers to things... I suppose I'll have to get over that at some point! |
Also, I don't know why I'm talking about persisting anything. The information is already in the metadata and can just be extracted from there. So the only challenge is making sure that all warning functions become somewhat contextual so that they can check their current class/field. That should be doable. |
Alright, if anyone wants to design these codes, you're very welcome to do so! Otherwise I'll merge this next week and we'll deal with that a bit later (but before the next release). |
I also want to add
@:haxe.warning
, but this isn't so easy because we probably have to persist this information in the types and fields so that all post-processing can check it. Some architectural challenges here.Syntax follows OCaml's somewhat:
We'll have to sort out the codes before the next release because we shouldn't really change these.