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

Clean up ctypes for names #1

Closed
gares opened this issue Sep 29, 2017 · 0 comments
Closed

Clean up ctypes for names #1

gares opened this issue Sep 29, 2017 · 0 comments
Assignees
Milestone

Comments

@gares
Copy link
Contributor

gares commented Sep 29, 2017

We currently use these 3 data types in the API

macro @gref :- ctype "Globnames.global_reference". % name for a global constant
macro @name :- ctype "Name.t".                     % name hint (compare as =)
string

I think we could add

macro @id :- ctype "Id.t"

and have all API adding things to the environment to require an @id, have conversion functions from the others to @id.

This new type could also be used in the module-related API, since @name is broken wrt comparison.

@gares gares added this to the v0.5 milestone Sep 29, 2017
@gares gares self-assigned this Sep 29, 2017
@gares gares closed this as completed Nov 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant