New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 719964 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 710441
Owner: ----
Closed: May 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS:
Pri: ----
Type: Bug



Sign in to add a comment

clang-format quality problem

Project Member Reported by pauljensen@chromium.org, May 9 2017

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
 
Labels: clang-format
Mergedinto: 710441
Status: Duplicate (was: Untriaged)

Sign in to add a comment