-
Notifications
You must be signed in to change notification settings - Fork 21
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
Having 2 functions with the same name should throw an error. #235
Comments
Yes, it definitely should; but right now there is zero semantic analysis going on. There is a sanity check that a function name doesn't clash with some important built-ins (like true, and yes this happened for real once, just remember all safety rules are written in blood) but other than that there is nothing going on: no symbol table no name resolution, nothing. I have a very big project ahead of me to do this, as it would enable a large number of useful checks (e.g. look at issues requiring sem that are open right now). I plan to do this; but don't expect anything this year... |
In fact I've re-opened #236 and put the same labels on it to make sure I capture this |
No rush at all. miss_hit as it is has already brought much joy and removed a lot of my pain from my every day life. So anything beyond that is just icing on the cake. 👍 |
That is some really nice feedback, it makes me very happy to hear that. Thanks! :) |
What kind of feature is this?
New feature in MISS_HIT
Your MATLAB/Octave environment
MATLAB
Octave is not affected by this
MISS_HIT component affected
I think this falls under the linter.
Describe the solution you'd like
Matlab will not use a file where 2 functions have the same name. I suspect that ms lint should flag this, no?
Example:
The text was updated successfully, but these errors were encountered: