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

google-java-format Eclipse plugin does not provide AOSP Style option as google-java-format IntelliJ plugin #179

Open
BruceZu opened this issue Aug 21, 2017 · 8 comments
Labels

Comments

@BruceZu
Copy link

BruceZu commented Aug 21, 2017

The google-java-format Eclipse plugin works in Eclipse IDE for Java Developers
Version: Oxygen Release (4.7.0)
Build id: 20170620-1800

But it does not provide AOSP Style option. Is this intentional design?
Most members of my team like AOSP style, while half of them are using Eclipse.
Is there a workaround to get over this problem. E.g. can you expose an XML configuration file for AOSP.
Thank you!

screenshot from 2017-08-20 21-51-25
screenshot from 2017-08-20 21-50-30

@sormuras
Copy link
Contributor

sormuras commented Aug 21, 2017

The easiest way is to install a second "Formatter implementation" plugin that provides AOSP style formatting out of the box. Then the user may choose the first or second implementation in the drop-down box shown in your screenshot.

@sormuras
Copy link
Contributor

This is the line that creates the Formatter with default options: https://github.com/google/google-java-format/blob/master/core/src/main/java/com/google/googlejavaformat/java/SnippetFormatter.java#L60
Here you could create a Formatter instance using AOSP style.

@BruceZu
Copy link
Author

BruceZu commented Aug 21, 2017

Thank you! @sormuras
About the idea to "install a second "Formatter implementation" plugin that provides AOSP style ".
I am wondering if it has the same effect as 'google-java-format IntelliJ plugin'. Because I just fetched a copy of https://github.com/android/platform_development/blob/master/ide/eclipse/android-formatting.xml and configured it in eclipse. Let Eclipse format a class and find the result is not same as that formated by 'google-java-format IntelliJ plugin'.

@BruceZu
Copy link
Author

BruceZu commented Aug 21, 2017

@sormuras
This is a good idea to add a hot patch. I want to know where I can get the src code of 'google-java-format Eclipse plugin'.
If I need to add a patch I think I should add it to the 'google-java-format Eclipse plugin'.
correct me if I am wrong. Thank you!

@sormuras
Copy link
Contributor

sormuras commented Aug 21, 2017

AFAIK, there's no (easy) possibility to present custom formatter options for the selected formatter implementation in Eclipse.

The Eclipse plugin contains an entire GJF distro -- using a default Formatter instance with default FormatterOptions.

This is what I'd do:

  • Fork the project and copy the eclipse_plugin folder to eclipse_plugin_aosp.
  • Modify the eclipse_plugin_aosp source to produce a different artifact (jar) then eclipse_plugin.
  • Enhance the SnippetFormatter to take a FormatterOptions instance.
  • Use/pass the normal options in eclipse_plugin and the AOSP style options eclipse_plugin_aosp

Finally, use https://jitpack.io to build and host your fork with two eclipse plugins.

@bryanschofield
Copy link

I've create a PR to address this: #251

@chkpnt
Copy link

chkpnt commented Jan 20, 2022

It is weird a solution to this issue doesn't seem to be needed. Looks like nobody is using Eclipse anymore and I'm the only one.

After playing around a little bit with google-java-format, I think for Eclipse users it is still more convenient to use the old eclipse-java-google-style.xml.

@ChristianCiach
Copy link

ChristianCiach commented Apr 26, 2022

I still would like to use the AOSP style with eclipse and I would greatly appreciate any kind of feedback from Google about the PR #251

The XML files for Eclipse do not behave the same as the actual formatter implementation. I see them more as an approximation of the real thing. I actually think that the XML files should be removed. See google/styleguide#687

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants