mCRL2org / mCRL2

The Git repository for the mCRL2 toolset.
https://mcrl2.org/
Boost Software License 1.0
88 stars 37 forks source link

Remove obsolete gsMake and gsIs functions. #1155

Closed jgroote closed 11 years ago

jgroote commented 11 years ago

Issue migrated from trac ticket # 1154

component: General | priority: major | resolution: fixed | keywords: gsMake and gsIs functions

2013-03-21 22:06:26: @jgroote created the issue


In order to remove obsolete gsMake and gsIs functions generated classes are required for the following objects:

gsMakeIdAssignment gsIsIdAssignment gsMakeParamId gsIsParamId gsMakeNil gsIsNil

gsMakeDataSpec gsMakeSortSpec gsMakeConsSpec gsMakeMapSpec gsMakeDataEqnSpec

gsMakeSetComp gsMakeBagComp gsMakeBagComp

jgroote commented 11 years ago

2013-03-25 09:59:08: @wiegerw changed status from new to closed

jgroote commented 11 years ago

2013-03-25 09:59:08: @wiegerw set resolution to fixed

jgroote commented 11 years ago

2013-03-25 09:59:08: @wiegerw commented


Three classes have been added in revision 11700:

IdAssignment -> process::id_assignment
ParamId      -> process::parameter_identifier
Nil          -> core::nil

The following classes were already present:

SetComp -> data::set_comprehension_binder
BagComp -> data::bag_comprehension_binder

It has been decided not to generate classes for SortSpec, ConsSpec, MapSpec and DataEqnSpec, since there is not a real need for them. In the case of SortSpec there is also a practical reason not to generate classes for it. SortSpec depends on SortDecl, which can be either a SortId or a SortRef. But SortId and SortRef are unrelated, so it does not make sense to generate this artificial dependency.

The class data_specification can not be generated from DataSpec, since little of the generated code can be reused in the manual implementation.

Finally, removal of gsMake and gsIs functions is of course desirable, but it does not have a high priority. I propose to do this after a number of problems in the internal format have been fixed.

jgroote commented 11 years ago

2013-08-23 15:51:10: scranen@win.tue.nl