Add clang-format preupload check.

This will check that new and changed lines conform to the style defined
in .clang-format. Specific code sections can be excluded if it gets in
the way (some examples included).

Also fix the java style to not line break in "/// CHECK" comments, and
use the AOSP style import order.

Java files in art/tools/ahat and art/tools/dexfuzz use indentation
width 2, so they get an override .clang-format with that style.

Java files in test/ have a mix of different indentation widths, so that
directory is excluded for the time being.

openjdkjvmti/include/jvmti.h is an imported file that should be ignored
altogether. However, it is not straightforward to configure a whole
file exclude. Upload it with --no-verify if it needs to be updated.

Test: clang-format -i art/dex2oat/dex2oat_options.cc \
        art/benchmark/const-class/src/ConstClassBenchmark.java \
        art/tools/ahat/src/main/com/android/ahat/heapdump/Parser.java \
        art/tools/dexfuzz/src/dexfuzz/program/IdCreator.java
  Check that reformatted files look reasonable
Test: Create a local commit with changes in both art/libartservice and
  art/test, run tools/repohooks/pre-upload.py, and verify that only the
  changes in art/libartservice get checked.
Bug: 181877164
Change-Id: I991ff032694bf5063e8516fdf71a95d0b047ffea

1 files changed, 2 insertions, 0 deletions

diff --git a/cmdline/cmdline_parser_test.cc b/cmdline/cmdline_parser_test.cc
index 38c37ec127..effbee93c7 100644
--- a/cmdline/cmdline_parser_test.cc
+++ b/cmdline/cmdline_parser_test.cc
@@ -528,10 +528,12 @@ TEST_F(CmdlineParserTest, TestVerify) {
 TEST_F(CmdlineParserTest, TestIgnoreUnrecognized) {
   RuntimeParser::Builder parserBuilder;
 
+  // clang-format off
   parserBuilder
       .Define("-help")
           .IntoKey(M::Help)
       .IgnoreUnrecognized(true);
+  // clang-format on
 
   parser_.reset(new RuntimeParser(parserBuilder.Build()));