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

Do not insert line breaks into function/event/hook definitions if they fit on a single line #7

Open
bbannier opened this issue Mar 17, 2022 · 0 comments
Labels
style tweak A suggestion to style things differently

Comments

@bbannier
Copy link
Member

bbannier commented Mar 17, 2022

As of zeek-format-1.1.0 definitions of e.g., events without body or with one-statement bodies always inserts a line break. Instead I would have expected no line break, at least in the case where there is no body (one could argue that with a body forcing a line break is good).

Example:

# Start point:
event zeek_init() {}

# Is formatted as:
event zeek_init()
{ } # Note forced ` ` in body.
@ckreibich ckreibich added the style tweak A suggestion to style things differently label May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
style tweak A suggestion to style things differently
Projects
None yet
Development

No branches or pull requests

2 participants