Skip to content

Unify AST with Smoosh #5

Description

@mgree

Right now ast.ml and shim.ml in smoosh are out of sync. It'd be great if there were only one copy of the code that converts dash's data structures to OCaml trees.

Some trickiness: it'd be ideal to not have to port over all of the smoosh runtime terms, just the actual parseable ones. Doing so would add an extra layer of conversion, though, which is probably pretty costly. It should be possible to functorize the dash conversion: given some functions that generate appropriate values of some abstract type, we'll call them in the right order.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestsmooshChanges here depend on changes in smoosh proper

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions