Indenting too much

[noel-yap]

The following code is indented too much after Array.of:

                  final Array<Integer> myArray =
                      Array.of(
                              …);

And since it's nowhere near the 100 character line limit, ideally it would be:

                  final Array<Integer> myArray = Array.of(
                      …);

[tbroyer]

Isn't it the Rectangle Rule at work?

[noel-yap]

I wasn't aware of that rule. Thanks for point me to it.

I'm thinking this what I had posted should be considered to be an exception to the rule. The following may fall under the same exception:

…
    .map(arg -> {
      return …;
    })

is, IMO, more readable than:

…
    .map(
        arg -> {
          return …;
        })

considering it's very similar to:

void map(int arg) {
  …
}

I'm wondering if the root issue is the preference towards exposing program structure over exposing the more meaningful parts of the program. For example, the body of the lambda passed into map provides more semantics to what's happening in the program than do the argument names. I would say the same holds true for the arguments to Array.of versus the Array.of itself.

[tbroyer]

See #19 about lambdas

[cushon]

@tbroyer covered it, the formatter prioritizes breaking at higher syntactic levels (i.e. the rectangle rule) over conserving vertical space or minimizing lines or indentation.