Add a .clang-format file to give us local control over Blink style.

Add a .clang-format file to give us local control over Blink style.

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.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=160157

[Jeffrey Yasskin, 2013-10-21 21:02:11]

Defining the style here will let us change it (within existing options) without
involving a clang commit, and it'll let editors pass just "-style=file
-assume-filename=<filename>" to clang-format instead of needing to guess the
desired style.

Please cc anyone else who should be involved.

https://codereview.chromium.org/32673005/diff/1/.clang-format
File .clang-format (right):

https://codereview.chromium.org/32673005/diff/1/.clang-format#newcode3
.clang-format:3: BasedOnStyle: WebKit
Instead of this line, I could use the full output of `clang-format -style=WebKit
-dump-config`.

[eseidel, 2013-10-21 21:33:13]

lgtm from a blink perspective.  You're certainly welcome to add a .file to the
top level directory if that makes clang-format happier.

[eseidel, 2013-10-21 21:33:38]

You could also add yourself as an OWNER of this file, assuming you plan to make
many more modifications to it.

[Jeffrey Yasskin, 2013-10-21 21:37:53]

On 2013/10/21 21:33:38, eseidel wrote:
> You could also add yourself as an OWNER of this file, assuming you plan to
make
> many more modifications to it.

Heh, no thanks. :) The people who own this file should be the people who control
overall Blink style, since changes to this file will change how clang-format
formats the code.

[eseidel, 2013-10-21 21:38:47]

So the immediate effect of this is that you no longer have to specify the style
when running clang-format under the blink checkout directory?

[Jeffrey Yasskin, 2013-10-21 21:49:20]

On 2013/10/21 21:38:47, eseidel wrote:
> So the immediate effect of this is that you no longer have to specify the
style
> when running clang-format under the blink checkout directory?

It's more that "-style=file" will work, and after I submit a similar change to
the Chromium repo, the same command-line option will work across both
directories. -style currently defaults to "LLVM", so you still have to specify
something to get Blink style.

Daniel, what do you think about making -style=file the default? (That shouldn't
block this change, I think.)

[commit-bot: I haz the power, 2013-10-21 22:32:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jyasskin@chromium.org/32673005/1

[commit-bot: I haz the power, 2013-10-22 08:55:36]

Change committed as 160157