-
Notifications
You must be signed in to change notification settings - Fork 194
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
Instantiations Wrapped with One Port #1889
Comments
Thanks for the report. Yes indeed, these things should be more configurable. So three choices would be good: multi-line always, single-line iff one parameter, single-line if all parameters fit within line length. @jbylicki Since you're on a roll currently with formatter updates, is this something that interests you ? |
Yeah to prevent bloat I think just two options would suffice: Multi-line always. single-line iff one parameter and/or port seems like a bug/side effect of some other rule. I don't know any style that would use this. |
Behavior around module instantiation wrapping doesn't seem desirable. Whenever only one port and/or parameter is given, the instantiation is done on one line (as long as it fits on the line). If 2 ports or 2 parameters are given, then the instantiation style is wrapped, even if it does fit on one line.
Test case
Actual output
Include any possible diagnostic messages from the formatter.
The output I desire
I always want to wrap my instantiations, even if it fits on one line.
Another Output Option that Some May Prefer
Some may prefer making the instantiation fit on one line if it is <= the character limit.
The text was updated successfully, but these errors were encountered: