Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 3692d9d4 authored by Ray Essick's avatar Ray Essick
Browse files

Cleaner logging in a 32/64 bit world

Use the portable PRId64 construct to print our 64-bit value.

Bug: 16890215
Test: compilation
Change-Id: If1b5d1731c8d2c4dc267889763ec04a4e3ac5e8e
parent 7857e9da
Loading
Loading
Loading
Loading
+2 −4
Original line number Diff line number Diff line
@@ -17,6 +17,7 @@
//#define LOG_NDEBUG 0
#define LOG_TAG "PluginMetricsReporting"
#include <utils/Log.h>
#include <inttypes.h>

#include <media/PluginMetricsReporting.h>

@@ -81,10 +82,7 @@ status_t reportMetricsGroup(const MetricsGroup& metricsGroup,

    analyticsItem.setFinalized(true);
    if (!analyticsItem.selfrecord()) {
      // Note the cast to int is because we build on 32 and 64 bit.
      // The cast prevents a peculiar printf problem where one format cannot
      // satisfy both.
      ALOGE("selfrecord() returned false. sessioId %d", (int) sessionId);
      ALOGE("selfrecord() returned false. sessioId %" PRId64, sessionId);
    }

    for (int i = 0; i < metricsGroup.metric_sub_group_size(); ++i) {