From 448fe573e079868db0fa4bf1e95bb9056888b511 Mon Sep 17 00:00:00 2001 From: Ruslan Lesiutin Date: Wed, 25 Jun 2025 05:38:31 -0700 Subject: [PATCH] Avoid potential copies of TraceEvent before serialization (#52196) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/52196 # Changelog: [Internal] Probably been overlooked for quite some time, but shouldn't be a bottleneck. Reviewed By: motiz88 Differential Revision: D77148271 fbshipit-source-id: e8eb32137086d6c280aab2ec5903be03f96175ad --- .../jsinspector-modern/tracing/PerformanceTracer.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/react-native/ReactCommon/jsinspector-modern/tracing/PerformanceTracer.cpp b/packages/react-native/ReactCommon/jsinspector-modern/tracing/PerformanceTracer.cpp index 0dc058f8967..07dd7b2624f 100644 --- a/packages/react-native/ReactCommon/jsinspector-modern/tracing/PerformanceTracer.cpp +++ b/packages/react-native/ReactCommon/jsinspector-modern/tracing/PerformanceTracer.cpp @@ -94,7 +94,7 @@ void PerformanceTracer::collectEvents( } auto traceEvents = folly::dynamic::array(); - for (auto event : buffer_) { + for (const auto& event : buffer_) { // Emit trace events traceEvents.push_back(serializeTraceEvent(event));