Skip to content
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

SANY should indicate if a module it uses is a built-in one #13

Closed
lemmy opened this issue Oct 7, 2016 · 1 comment
Closed

SANY should indicate if a module it uses is a built-in one #13

lemmy opened this issue Oct 7, 2016 · 1 comment
Labels
enhancement Lets change things for the better Tools The command line tools - TLC, SANY, ...

Comments

@lemmy
Copy link
Member

lemmy commented Oct 7, 2016

xxyzzn:

A flag should be aded to modules which are built-in and are loaded by SANY from a specific folder. This information is required for the PM in order to overide the symbols. For example, the Integer.tla module should, if loaded from the pre-configured location, be labelled with "dummy=true" or "build-in=true" in the SANY syntax tree.

libal:

I just confirmed that the SANY parser reports the operators in Naturals, TLAPS and other library modules with kind UserDefinedOpKind. Since these operators have fixed semantics, it would be nice to have them under kind BuiltInKind.

xxyzzn:

The operators in Naturals, etc. are not built-in operators, so it would be very un-nice if SANY reported them as such. For example, it would make it impossible for the user to have TLC override the definition of the operator Seq from the Sequences module.

xxyzzn:

A field named isStandard that indicates if the module is a standard module was added to the ModuleNode class in the version of the code I just committed.

@lemmy lemmy added enhancement Lets change things for the better Tools The command line tools - TLC, SANY, ... labels Oct 7, 2016
@lemmy
Copy link
Member Author

lemmy commented May 17, 2018

Do not see an actionable item, thus closing. Please reopen if necessary.

@lemmy lemmy closed this as completed May 17, 2018
quaeler added a commit that referenced this issue May 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Lets change things for the better Tools The command line tools - TLC, SANY, ...
Projects
None yet
Development

No branches or pull requests

1 participant