src: remove unnecessary copy operations in tracing

PR-URL: #20356 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
nodejs · May 4, 2018 · 31812ed · 31812ed
1 parent 3962c73
commit 31812ed
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/node_trace_events.cc b/src/node_trace_events.cc
@@ -63,7 +63,7 @@ void NodeCategorySet::Enable(const FunctionCallbackInfo<Value>& args) {
   NodeCategorySet* category_set;
   ASSIGN_OR_RETURN_UNWRAP(&category_set, args.Holder());
   CHECK_NE(category_set, nullptr);
-  auto categories = category_set->GetCategories();
+  const auto& categories = category_set->GetCategories();
   if (!category_set->enabled_ && !categories.empty()) {
     env->tracing_agent()->Enable(categories);
     category_set->enabled_ = true;
@@ -75,7 +75,7 @@ void NodeCategorySet::Disable(const FunctionCallbackInfo<Value>& args) {
   NodeCategorySet* category_set;
   ASSIGN_OR_RETURN_UNWRAP(&category_set, args.Holder());
   CHECK_NE(category_set, nullptr);
-  auto categories = category_set->GetCategories();
+  const auto& categories = category_set->GetCategories();
   if (category_set->enabled_ && !categories.empty()) {
     env->tracing_agent()->Disable(categories);
     category_set->enabled_ = false;