You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In lbl-srg/modelica-buildings#2426, I applied modelicafmt commit 325ef64 with a few tweaks. I suggest these to be added to modelicafmt. The script at the above PR shows the changes, these are
If a function has only one argument, then don't do a line break. The reason is that we have many functions such as pre(), der() and others (sin(), ...) that are better to keep on one line.
Add an extra line break before equation and initial equation, or initial algorithm and algorithm to improve readability.
In the revision notes of the .mo code, there is an extra empty line after the last </ul>. E.g., the file has </ul>\n\n</html> rather than </ul>\n</html>.
The text was updated successfully, but these errors were encountered:
In lbl-srg/modelica-buildings#2426, I applied
modelicafmt
commit 325ef64 with a few tweaks. I suggest these to be added tomodelicafmt
. The script at the above PR shows the changes, these arepre()
,der()
and others (sin()
, ...) that are better to keep on one line.equation
andinitial equation
, orinitial algorithm
andalgorithm
to improve readability..mo
code, there is an extra empty line after the last</ul>
. E.g., the file has</ul>\n\n</html>
rather than</ul>\n</html>
.The text was updated successfully, but these errors were encountered: