search

modelica-tools / atom-language-modelica

Modelica language support for the atom editor
https://atom.io/packages/language-modelica
Other
8 stars 4 forks source link

Modelica keywords #19

Closed lochel closed 8 years ago

lochel commented 8 years ago

I think the keywords are not complete, initial algorithm is missing at least. Also I am wondering why register is in the keyword list.

Is there more to fix?

lochel commented 8 years ago

I looked up all keywords and these are missing: encapsulated, external, impure, in, inner, operator, outer, pure, return, and type.

thorade commented 8 years ago

I did not know pure and impure were in the Spec already, but if you looked it up then you are informed much better. Is having initial, equation and algorithm sufficient or do composite keywords like initial algorithm have to be listed separately? If there is no further discussion I would just accept your PR tomorrow (or you can do it yourself).

tshort commented 8 years ago

+1

lochel commented 8 years ago

initial is listed additionally as composite keyword since it should be displayed differently if it is used as function call or preamble of an initial equation/algorithm section.

thorade commented 8 years ago

Who will run apm publish patch ?

thorade commented 8 years ago

OK, I published v0.3.1