summaryrefslogtreecommitdiffstats
path: root/runtime
diff options
context:
space:
mode:
authorBrian Carlstrom <bdc@google.com>2013-07-18 18:21:14 -0700
committerBrian Carlstrom <bdc@google.com>2013-07-18 21:55:45 -0700
commitfb6996fe2b1bb28526751650441d7ac29dcdb4e6 (patch)
treeaba94b367c89719413d4be8fa7fd445ce3bee743 /runtime
parent3e79aadc447dcdb9e52368baced9405667e68e6a (diff)
downloadart-fb6996fe2b1bb28526751650441d7ac29dcdb4e6.zip
art-fb6996fe2b1bb28526751650441d7ac29dcdb4e6.tar.gz
art-fb6996fe2b1bb28526751650441d7ac29dcdb4e6.tar.bz2
Fixing cpplint runtime/arrays, runtime/int, runtime/virtual issues
Change-Id: Ia2ff94d2fb69465df26aaf83df82614a483b26e0
Diffstat (limited to 'runtime')
-rw-r--r--runtime/base/mutex.cc6
-rw-r--r--runtime/jdwp/jdwp_event.cc2
-rw-r--r--runtime/mirror/class.h6
3 files changed, 7 insertions, 7 deletions
diff --git a/runtime/base/mutex.cc b/runtime/base/mutex.cc
index af45f3f..cb66240 100644
--- a/runtime/base/mutex.cc
+++ b/runtime/base/mutex.cc
@@ -32,7 +32,7 @@ namespace art {
// This works on Mac OS 10.6 but hasn't been tested on older releases.
struct __attribute__((__may_alias__)) darwin_pthread_mutex_t {
- long padding0;
+ long padding0; // NOLINT(runtime/int)
int padding1;
uint32_t padding2;
int16_t padding3;
@@ -43,7 +43,7 @@ struct __attribute__((__may_alias__)) darwin_pthread_mutex_t {
};
struct __attribute__((__may_alias__)) darwin_pthread_rwlock_t {
- long padding0;
+ long padding0; // NOLINT(runtime/int)
pthread_mutex_t padding1;
int padding2;
pthread_cond_t padding3;
@@ -72,7 +72,7 @@ struct __attribute__((__may_alias__)) glibc_pthread_rwlock_t {
#if ART_USE_FUTEXES
static bool ComputeRelativeTimeSpec(timespec* result_ts, const timespec& lhs, const timespec& rhs) {
- const long int one_sec = 1000 * 1000 * 1000; // one second in nanoseconds.
+ const int32_t one_sec = 1000 * 1000 * 1000; // one second in nanoseconds.
result_ts->tv_sec = lhs.tv_sec - rhs.tv_sec;
result_ts->tv_nsec = lhs.tv_nsec - rhs.tv_nsec;
if (result_ts->tv_nsec < 0) {
diff --git a/runtime/jdwp/jdwp_event.cc b/runtime/jdwp/jdwp_event.cc
index 52dd782..546c637 100644
--- a/runtime/jdwp/jdwp_event.cc
+++ b/runtime/jdwp/jdwp_event.cc
@@ -1046,7 +1046,7 @@ void JdwpState::DdmSendChunkV(uint32_t type, const iovec* iov, int iov_count) {
* "Wrap" the contents of the iovec with a JDWP/DDMS header. We do
* this by creating a new copy of the vector with space for the header.
*/
- iovec wrapiov[iov_count+1];
+ iovec wrapiov[iov_count+1]; // NOLINT(runtime/arrays) iov_count < 10
for (int i = 0; i < iov_count; i++) {
wrapiov[i+1].iov_base = iov[i].iov_base;
wrapiov[i+1].iov_len = iov[i].iov_len;
diff --git a/runtime/mirror/class.h b/runtime/mirror/class.h
index 9a506c2..1dd02c0 100644
--- a/runtime/mirror/class.h
+++ b/runtime/mirror/class.h
@@ -27,10 +27,10 @@
* [This is an unlikely "natural" value, since it would be 30 non-ref instance
* fields followed by 2 ref instance fields.]
*/
-#define CLASS_WALK_SUPER ((unsigned int)(3))
-#define CLASS_BITS_PER_WORD (sizeof(unsigned long int) * 8)
+#define CLASS_WALK_SUPER 3U
+#define CLASS_BITS_PER_WORD (sizeof(uint32_t) * 8)
#define CLASS_OFFSET_ALIGNMENT 4
-#define CLASS_HIGH_BIT ((unsigned int)1 << (CLASS_BITS_PER_WORD - 1))
+#define CLASS_HIGH_BIT (1U << (CLASS_BITS_PER_WORD - 1))
/*
* Given an offset, return the bit number which would encode that offset.
* Local use only.