commit b1eba213afaf7fa6445de863ddc9680ab99762ea
author Brian Carlstrom <bdc@google.com> Wed Jul 17 18:07:19 2013 -0700
committer Brian Carlstrom <bdc@google.com> Wed Jul 17 21:46:11 2013 -0700
tree c0a3000df1037211a6bcc7d75db51ee5c6069ec8
parent 2ce745c06271d5223d57dbf08117b20d5b60694a

Fix cpplint whitespace/comma issues

Change-Id: I456fc8d80371d6dfc07e6d109b7f478c25602b65

runtime/base/macros.h

diff --git a/runtime/base/macros.h b/runtime/base/macros.h
index 4a196f2..44a7f1e 100644
--- a/runtime/base/macros.h
+++ b/runtime/base/macros.h
@@ -125,7 +125,7 @@
 #define OFFSETOF_VOLATILE_MEMBER(t, f) \
   (reinterpret_cast<volatile char*>(&reinterpret_cast<t*>(16)->f) - reinterpret_cast<volatile char*>(16)) // NOLINT
 
-#define PACKED(x) __attribute__ ((__aligned__(x),__packed__))
+#define PACKED(x) __attribute__ ((__aligned__(x), __packed__))
 
 #define LIKELY(x)       __builtin_expect((x), true)
 #define UNLIKELY(x)     __builtin_expect((x), false)

runtime/gc/allocator/dlmalloc.cc

diff --git a/runtime/gc/allocator/dlmalloc.cc b/runtime/gc/allocator/dlmalloc.cc
index 7584b6e..7725215 100644
--- a/runtime/gc/allocator/dlmalloc.cc
+++ b/runtime/gc/allocator/dlmalloc.cc
@@ -27,7 +27,7 @@
 static void art_heap_corruption(const char* function);
 static void art_heap_usage_error(const char* function, void* p);
 #define CORRUPTION_ERROR_ACTION(m) art_heap_corruption(__FUNCTION__)
-#define USAGE_ERROR_ACTION(m,p) art_heap_usage_error(__FUNCTION__, p)
+#define USAGE_ERROR_ACTION(m, p) art_heap_usage_error(__FUNCTION__, p)
 
 // Ugly inclusion of C file so that ART specific #defines configure dlmalloc for our use for
 // mspaces (regular dlmalloc is still declared in bionic).

runtime/runtime.cc

diff --git a/runtime/runtime.cc b/runtime/runtime.cc
index 14d4c8a..ba18311 100644
--- a/runtime/runtime.cc
+++ b/runtime/runtime.cc
@@ -709,7 +709,7 @@
 // Do zygote-mode-only initialization.
 bool Runtime::InitZygote() {
   // zygote goes into its own process group
-  setpgid(0,0);
+  setpgid(0, 0);
 
   // See storage config details at http://source.android.com/tech/storage/
   // Create private mount namespace shared by all children

runtime/thread.cc

diff --git a/runtime/thread.cc b/runtime/thread.cc
index 0020d52..269d58e 100644
--- a/runtime/thread.cc
+++ b/runtime/thread.cc
@@ -626,7 +626,7 @@
         ScopedLocalRef<jstring> scoped_name_string(env,
                                                    (jstring)env->GetObjectField(peer,
                                                               WellKnownClasses::java_lang_Thread_name));
-        ScopedUtfChars scoped_name_chars(env,scoped_name_string.get());
+        ScopedUtfChars scoped_name_chars(env, scoped_name_string.get());
         if (scoped_name_chars.c_str() == NULL) {
             LOG(WARNING) << "No such thread for suspend: " << peer;
             env->ExceptionClear();

runtime/verifier/reg_type.h

diff --git a/runtime/verifier/reg_type.h b/runtime/verifier/reg_type.h
index 1553f1e..c66e7cb 100644
--- a/runtime/verifier/reg_type.h
+++ b/runtime/verifier/reg_type.h
@@ -889,7 +889,7 @@
   UnresolvedMergedType(uint16_t left_id, uint16_t right_id, const RegTypeCache* reg_type_cache,
                        uint16_t cache_id)
       SHARED_LOCKS_REQUIRED(Locks::mutator_lock_)
-      : UnresolvedType("", cache_id), reg_type_cache_(reg_type_cache) ,merged_types_(left_id, right_id) {
+      : UnresolvedType("", cache_id), reg_type_cache_(reg_type_cache), merged_types_(left_id, right_id) {
     if (kIsDebugBuild) {
       CheckInvariants();
     }