Need empty line before first member definition of a class

[jjqq2013]

Thanks for your great work. I works well except some inconsistent output,

The output of following two java code are supposed to be same, but actually not, the both output the same as input.

public class Keys {
  private static class Identities0 {
    public static Identity<Long> IDENTITY_BILLING_OFFERINGS1 = Internal.create();
    public static Identity<Long> IDENTITY_BILLING_OFFERINGS2 = Internal.create();
  }
}

public class Keys {

  private static class Identities0 {

    public static Identity<Long> IDENTITY_BILLING_OFFERINGS1 = Internal.create();
    public static Identity<Long> IDENTITY_BILLING_OFFERINGS2 = Internal.create();
  }
}

I've tested this with version 1.8 and 1.7.

[tbroyer]

The Google Java Style says:

A blank line before the first member or initializer, or after the last member or initializer of the class, is neither encouraged nor discouraged.

(Source: https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace)

so it looks like google-java-format works as expected here, or at least it's not wrong.

[kevinb9n]

We often still have the formatter standardize on a single choice anyway, but in this case we decided not to. There are big long hairy signatures for which it really helps to use a blank line after; whereas the second snippet above shows why a policy of always using the blank line is sad.

[jjqq2013]

Thank you all for the comments. How do you consider the consistency? Essentially same java source files, generated difference output, though trivial?

[kuchaguangjie]

It's ugly & not neat to have that 2 empty lines.