| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
|
| |
The number of macros is too damn high!
BUG=373340
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/1235153009
Cr-Commit-Position: refs/heads/master@{#339216}
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use the new MacroBlockBegin/MacroBlockEnd options introduced in
http://reviews.llvm.org/rL241363 so clang-format can correctly indent
the bodies of IPC message maps, etc.
BUG=373340
R=thakis@chromium.org
TBR=darin@chromium.org
Review URL: https://codereview.chromium.org/1225333004 .
Cr-Commit-Position: refs/heads/master@{#339131}
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Those spaces are explicitly optional for Google's Java style (Point 8 of
https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s4.6.2-horizontal-whitespace)
and not having spaces outnumbers spaces about 10-to-1 in our existing code.
This also removes the space in annotations like `@Feature({"Android-AppBase"})`
which is too looks like a good change.
With this, all chromium-related java bits are upstreamed into clang-format and
the java-specific section from .clang-format is gone again.
BUG=429372
NOTRY=true
Review URL: https://codereview.chromium.org/764423002
Cr-Commit-Position: refs/heads/master@{#306245}
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before:
void f() {
}
int g() {
return 0;
}
Now:
void f() {}
int g() {
return 0;
}
Do this by removing the explicit setting for AllowShortFunctionsOnSingleLine,
as of recently [1] it defaults to "Empty" for Java, a new setting that does
what we want.
Chromium style used to unconditionally override AllowShortFunctionsOnSingleLine
to the wrong thing, so I taught clang-format to not do that for Java files, and
while there also made -style=Chromium do the right thing for most of the
settings [2], so remove most explicit settings from this file. Except for
AllowShortFunctionsOnSingleLine, this doesn't change behavior.
1: http://llvm.org/viewvc/llvm-project?view=revision&revision=222819
2: http://llvm.org/viewvc/llvm-project?view=revision&revision=222839
BUG=429372
NOTRY=true
Review URL: https://codereview.chromium.org/757313002
Cr-Commit-Position: refs/heads/master@{#305830}
|
| |
|
|
|
|
|
|
|
| |
BUG=429372
NOTRY=true
Review URL: https://codereview.chromium.org/755313002
Cr-Commit-Position: refs/heads/master@{#305657}
|
| |
|
|
|
|
|
|
|
|
|
| |
param" for Java.
BUG=429372
NOTRY=true
Review URL: https://codereview.chromium.org/724613004
Cr-Commit-Position: refs/heads/master@{#304923}
|
| |
|
|
|
|
|
|
|
| |
BUG=429372
R=aurimas@chromium.org
Review URL: https://codereview.chromium.org/727593002
Cr-Commit-Position: refs/heads/master@{#304122}
|
| |
|
|
|
|
|
|
|
|
|
| |
Based on formatting a few java files in base/ and comparing before/after.
BUG=429372
NOTRY=true
Review URL: https://codereview.chromium.org/697893002
Cr-Commit-Position: refs/heads/master@{#302406}
|
| |
|
|
|
|
|
|
|
| |
BUG=429372
NOTRY=true
Review URL: https://codereview.chromium.org/697783002
Cr-Commit-Position: refs/heads/master@{#302358}
|
| |
|
|
|
|
|
|
|
| |
BUG=428816
NOTRY=true
Review URL: https://codereview.chromium.org/696843002
Cr-Commit-Position: refs/heads/master@{#302341}
|
| |
|
|
|
|
|
|
|
|
|
| |
Per bug discussion, add Standard: Cpp11 to encourage ">>" over "> >".
TBR=thakis@chromium.org
BUG=417296
Review URL: https://codereview.chromium.org/599093008
Cr-Commit-Position: refs/heads/master@{#296665}
|
|
|
This also makes editor integration somewhat easier by letting editors pass just
"-style=file -assume-filename=<filename>" to clang-format instead of needing to
guess the desired style based on the path.
We could name this file _clang-format instead, if that's easier for Windows.
Review URL: https://codereview.chromium.org/52093012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232271 0039d316-1c4b-4281-b951-d872f2087c98
|
