Merge "Fix misc-macro-parentheses warnings in common_time."

This commit is contained in:
Treehugger Robot
2016-05-24 19:39:17 +00:00
committed by Gerrit Code Review
2 changed files with 7 additions and 7 deletions

View File

@@ -285,7 +285,7 @@ void CommonTimeServer::reevaluateAutoDisableState(bool commonClockHasClients) {
if (res > 0) \
write(fd, buffer, res); \
} while (0)
#define checked_percentage(a, b) ((0 == b) ? 0.0f : ((100.0f * a) / b))
#define checked_percentage(a, b) ((0 == (b)) ? 0.0f : ((100.0f * (a)) / (b)))
status_t CommonTimeServer::dumpClockInterface(int fd,
const Vector<String16>& /* args */,

View File

@@ -48,12 +48,12 @@ const uint16_t TimeServicePacketHeader::kCurVersion = 1;
#define SERIALIZE_INT32(field_name) SERIALIZE_FIELD(field_name, int32_t, htonl)
#define SERIALIZE_INT64(field_name) SERIALIZE_FIELD(field_name, int64_t, htonq)
#define DESERIALIZE_FIELD(field_name, type, converter) \
do { \
if ((offset + sizeof(field_name)) > length) \
return -1; \
field_name = converter(*((type*)(data + offset))); \
offset += sizeof(field_name); \
#define DESERIALIZE_FIELD(field_name, type, converter) \
do { \
if ((offset + sizeof(field_name)) > length) \
return -1; \
(field_name) = converter(*((type*)(data + offset))); \
offset += sizeof(field_name); \
} while (0)
#define DESERIALIZE_INT16(field_name) DESERIALIZE_FIELD(field_name, int16_t, ntohs)
#define DESERIALIZE_INT32(field_name) DESERIALIZE_FIELD(field_name, int32_t, ntohl)