Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tweak output of modelicafmt #26

Open
mwetter opened this issue Mar 31, 2021 · 0 comments
Open

Tweak output of modelicafmt #26

mwetter opened this issue Mar 31, 2021 · 0 comments

Comments

@mwetter
Copy link

mwetter commented Mar 31, 2021

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

  1. 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.
  2. Add an extra line break before equation and initial equation, or initial algorithm and algorithm to improve readability.
  3. 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>.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant