Conversation
| @Check(CheckType.FAST) | ||
| public void checkAction(Action action) { | ||
| checkName(action.getName(), Literals.VARIABLE__NAME); | ||
| checkName(action.getName(), Literals.MEMBER__NAME); |
There was a problem hiding this comment.
Shouldn't this PR also add checkName calls to other syntactic elements like method, reaction, parameters and state variables? Or am I missing something here about how this code works?
There was a problem hiding this comment.
I created an interface called Member that is implemented by all of those.
There was a problem hiding this comment.
Yes, but I expected to see a new check checkMember added that actually performs the name check. The way it looks now, we only check names for actions and ports. Or does Xtext automatically take care of checking uniqueness here?
There was a problem hiding this comment.
Uniqueness is checked automatically based on the name attribute. This only specifies which nodes should be checked against one another.
2 participants
This is towards addressing #1973.