src: remove unnecessary copy operations in tracing

PR-URL: #20356 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]>
nodejs · Apr 28, 2018 · 7016ee6 · 7016ee6
1 parent 16aee38
commit 7016ee6
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;