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

normalize formatting using default idea settings #51

Merged
merged 1 commit into from
May 24, 2016

Conversation

jdillon
Copy link
Contributor

@jdillon jdillon commented May 24, 2016

based on IDEA 2016 default profile (I think).

I did disable javadoc comment formatting else the license headers (which are presently javadocs) get reformatted. Header comment should probably be changed to multi-line single-star comment instead of javadoc double-star comment to avoid problems.

I left imports asis, default configuration seems to want to use * imports and I'm not sure that is what you really want. There are at least a few unused imports that could be removed.

There are also a few other turds I noticed like unneeded ';' in a few places but I left those alone for now too.

}
}

;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

example of uneeded ';' should be removed

@chirino chirino merged commit 03e7a2a into fusesource:master May 24, 2016
@chirino
Copy link
Member

chirino commented May 24, 2016

Thanks!

gnodet added a commit to gnodet/jansi that referenced this pull request Oct 5, 2021
gnodet added a commit that referenced this pull request Oct 13, 2021
@gnodet gnodet modified the milestone: 2.4.0 Oct 13, 2021
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

Successfully merging this pull request may close these issues.

3 participants