Skip to content

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

@BruceZu

Description

@BruceZu

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

Activity

sormuras

sormuras commented on Aug 21, 2017

@sormuras
Contributor

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

sormuras commented on Aug 21, 2017

@sormuras
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

BruceZu commented on Aug 21, 2017

@BruceZu
Author

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

BruceZu commented on Aug 21, 2017

@BruceZu
Author

@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

sormuras commented on Aug 21, 2017

@sormuras
Contributor

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

bryanschofield commented on Feb 6, 2018

@bryanschofield

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

chkpnt

chkpnt commented on Jan 20, 2022

@chkpnt

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

ChristianCiach commented on Apr 26, 2022

@ChristianCiach

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @cushon@BruceZu@chkpnt@sormuras@bryanschofield

        Issue actions

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