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

Consolidation of records and environments data structure #71

Open
jeromesimeon opened this issue Sep 8, 2017 · 0 comments
Open

Consolidation of records and environments data structure #71

jeromesimeon opened this issue Sep 8, 2017 · 0 comments

Comments

@jeromesimeon
Copy link
Member

The main data structure for both records and environments are bindings (in Utils/Bindings.v). A couple of things could be improved:

  • The internal representation (association lists, sorted without duplicates) bleeds through the rest of the code, and could be hidden better.
  • There are two different kinds of lookups (lookup and lookupr) used in various parts of the code. It's unclear whether this is a necessary distinction (records vs environments?) or something that could be eliminated.
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