saser / cakeml

CakeML: A Verified Implementation of ML
https://cakeml.org
Other
0 stars 0 forks source link

Use a `lang` JSON property for same-named constructors, instead of naming conventions #16

Closed saser closed 7 years ago

saser commented 7 years ago

We (currently) have two presLang constructors -- ModCon and ConCon -- for the two different Con constructors from modLang and conLang, respectively. When converting from presLang to JSON the object either has

{
    "cons": "Con-modLang"
    ...
}

or

{
    "cons": "Con-conLang"
    ...
}

respectively.

Instead, we should give it a lang property, resulting in

{
    "cons": "Con",
    "lang": "modLang"
    ...
}

or

{
    "cons": "Con",
    "lang": "conLang"
    ...
}

respectively.

hjorthjort commented 7 years ago

Will be fixed by API change

hjorthjort commented 7 years ago

Closed by #22