commit 8c3fc80af9c0529b4b4b0459fd70ddc203eb677c
author TreeHugger Robot <treehugger-gerrit@google.com> Wed Feb 02 19:30:02 2022 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Wed Feb 02 19:30:02 2022 +0000
tree a702a31578ed00900a4c26da2e1effaf187d0dcf
parent f3fa3413ae29862169f0fd4846b4580d5ec095a6
parent 6946d41e15119f85358f41e90fae1f77a71193ff

Merge "Avoid a couple string copies"

drm/libmediadrm/DrmMetricsConsumer.cpp

diff --git a/drm/libmediadrm/DrmMetricsConsumer.cpp b/drm/libmediadrm/DrmMetricsConsumer.cpp
index dca3050..c06f09b 100644
--- a/drm/libmediadrm/DrmMetricsConsumer.cpp
+++ b/drm/libmediadrm/DrmMetricsConsumer.cpp
@@ -32,7 +32,7 @@
 
 namespace {
 
-std::string GetAttributeName(std::string typeName, uint32_t attribute) {
+std::string GetAttributeName(const std::string &typeName, uint32_t attribute) {
     if (typeName == "KeyStatusChange") {
         static const char *type_names[] = {"USABLE", "EXPIRED",
                                        "OUTPUT_NOT_ALLOWED", "STATUS_PENDING",
@@ -85,7 +85,7 @@
 
 template <typename T>
 void ExportCounterMetricWithAttributeNames(
-    const android::CounterMetric<T> &counter, std::string typeName, PersistableBundle *metrics) {
+    const android::CounterMetric<T> &counter, const std::string &typeName, PersistableBundle *metrics) {
     if (!metrics) {
         ALOGE("metrics was unexpectedly null.");
         return;