diff --git a/libstats/push_compat/Android.bp b/libstats/push_compat/Android.bp index 819066eb4..c5c193495 100644 --- a/libstats/push_compat/Android.bp +++ b/libstats/push_compat/Android.bp @@ -26,7 +26,7 @@ package { cc_defaults { name: "libstatspush_compat_defaults", srcs: [ - "statsd_writer.c", + "statsd_writer.cpp", "stats_event_list.c", "StatsEventCompat.cpp" ], diff --git a/libstats/push_compat/statsd_writer.c b/libstats/push_compat/statsd_writer.cpp similarity index 98% rename from libstats/push_compat/statsd_writer.c rename to libstats/push_compat/statsd_writer.cpp index 4818d1112..a3600f322 100644 --- a/libstats/push_compat/statsd_writer.c +++ b/libstats/push_compat/statsd_writer.cpp @@ -15,9 +15,9 @@ */ #include "statsd_writer.h" +#include #include #include -#include #include #include #include @@ -108,7 +108,7 @@ static int statsdOpen() { case -ECONNREFUSED: case -ENOENT: i = atomic_exchange(&statsdLoggerWrite.sock, ret); - /* FALLTHRU */ + break; default: break; } @@ -188,7 +188,7 @@ static int statsdWrite(struct timespec* ts, struct iovec* vec, size_t nr) { * }; */ - header.tid = gettid(); + header.tid = android::base::GetThreadId(); header.realtime.tv_sec = ts->tv_sec; header.realtime.tv_nsec = ts->tv_nsec; @@ -272,7 +272,7 @@ static int statsdWrite(struct timespec* ts, struct iovec* vec, size_t nr) { if (ret < 0) { ret = -errno; } - /* FALLTHRU */ + break; default: break; } diff --git a/libstats/push_compat/statsd_writer.h b/libstats/push_compat/statsd_writer.h index fe2d37cbc..f030b96c7 100644 --- a/libstats/push_compat/statsd_writer.h +++ b/libstats/push_compat/statsd_writer.h @@ -21,6 +21,8 @@ #include #include +__BEGIN_DECLS + /** * Internal lock should not be exposed. This is bad design. * TODO: rewrite it in c++ code and encapsulate the functionality in a @@ -42,4 +44,6 @@ struct android_log_transport_write { void (*noteDrop)(int error, int tag); }; +__END_DECLS + #endif // ANDROID_STATS_LOG_STATS_WRITER_H