Hello Alexey Serbin, Attila Bukor, Kudu Jenkins, Grant Henke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16260
to look at the new patch set (#2).
Change subject: Add .clang-format file
......................................................................
Add .clang-format file
This clang-format file seems to capture our existing style relatively
well. I've been using it on my patches and only gotten a few complaints
where the automatic formatting isn't quite what people expected, but I
think the benefits of uniformity outweigh any particular aesthetic
choices we might make.
This patch updates the documentation to suggest running this on patches
to modify only the changed lines using clang-format-diff.
Change-Id: I41def9a77bd98bf09353fe9a7789756a1bff30c1
---
A build-support/clang_format_diff.sh
M docs/contributing.adoc
A src/kudu/.clang-format
3 files changed, 94 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/16260/2
--
To view, visit http://gerrit.cloudera.org:8080/16260
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I41def9a77bd98bf09353fe9a7789756a1bff30c1
Gerrit-Change-Number: 16260
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
