Issue metadata
Sign in to add a comment
|
clang-format quality problem |
||||||||||||||||||||||
Issue description
clang-format produced code that (choose all that apply):
Probably violates Chromium coding style.
- No sane human would ever choose
Here's the code before formatting:
class a {
void f() {
new Thread(new Runnable() {
public void run() {}
}).start();
}
}
Here's the code after formatting:
class a {
void f() {
new Thread(new Runnable() {
public void run() {}
})
.start();
}
}
Here's how it ought to look:
class a {
void f() {
new Thread(new Runnable() {
public void run() {}
}).start();
}
}
Code review link for full files/context:
https://codereview.chromium.org/2872593002/diff/20001/components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java?context=10&column_width=100&tab_spaces=8
,
May 9 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by thakis@chromium.org
, May 9 2017