Java 8 Method References are not formatted correctly |
||
Issue description
clang-format produced code that (choose all that apply):
+ Doesn't match Chromium style
+ Riles my finely honed stylistic dander
+ No sane human would ever choose
Seems like Java 8 Method References are given extra spaces for no reason.
Here's the code before formatting:
private void foo() {
someFunctionAcceptingALambda(this::methodReference);
}
Here's the code after formatting:
private void foo() {
someFunctionAcceptingALambda(this ::methodReference);
}
Here's how it ought to look:
- no change -
Code review link for full files/context:
https://chromium-review.googlesource.com/c/chromium/src/+/1227216/2/chrome/android/java/src/org/chromium/chrome/browser/explore_sites/ExperimentalExploreSitesSection.java
,
Oct 3
,
Nov 26
A clang-format with the fix got deployed on Nov 21 2018. |
||
►
Sign in to add a comment |
||
Comment 1 by thakis@chromium.org
, Sep 16Summary: Java 8 Method References are not formatted correctly (was: clang-format quality problem)