Skip to content

Commit 3cf11a3

Browse files
hoxyqfacebook-github-bot
authored andcommitted
Flatten struct (#52867)
Summary: Pull Request resolved: #52867 # Changelog: [Internal] There are currently no reasons for these struct not to be plain, plus this would allow us avoiding potentially expensive copying, when returning `const &`. Reviewed By: huntie Differential Revision: D78919222 fbshipit-source-id: 7b39d754c05b25915f07202d7e4839b10a08a47c
1 parent 4023800 commit 3cf11a3

File tree

3 files changed

+5
-11
lines changed

3 files changed

+5
-11
lines changed

packages/react-native/ReactCommon/jsinspector-modern/InstanceAgent.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,9 @@ tracing::InstanceTracingProfile InstanceAgent::collectTracingProfile() {
172172
tracing::RuntimeSamplingProfile runtimeSamplingProfile =
173173
runtimeAgent_->collectSamplingProfile();
174174

175-
return tracing::InstanceTracingProfile{std::move(runtimeSamplingProfile)};
175+
return tracing::InstanceTracingProfile{
176+
.runtimeSamplingProfile = std::move(runtimeSamplingProfile),
177+
};
176178
}
177179

178180
} // namespace facebook::react::jsinspector_modern

packages/react-native/ReactCommon/jsinspector-modern/TracingAgent.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ bool TracingAgent::handleRequest(const cdp::PreparsedRequest& req) {
115115
dataCollectedCallback,
116116
PROFILE_TRACE_EVENT_CHUNK_SIZE);
117117
serializer.serializeAndNotify(
118-
instanceAgent_->collectTracingProfile().getRuntimeSamplingProfile(),
118+
instanceAgent_->collectTracingProfile().runtimeSamplingProfile,
119119
instanceTracingStartTimestamp_);
120120

121121
frontendChannel_(cdp::jsonNotification(

packages/react-native/ReactCommon/jsinspector-modern/tracing/InstanceTracingProfile.h

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,7 @@ namespace facebook::react::jsinspector_modern::tracing {
1313

1414
struct InstanceTracingProfile {
1515
public:
16-
explicit InstanceTracingProfile(RuntimeSamplingProfile runtimeSamplingProfile)
17-
: runtimeSamplingProfile_(std::move(runtimeSamplingProfile)) {}
18-
19-
const RuntimeSamplingProfile& getRuntimeSamplingProfile() const {
20-
return runtimeSamplingProfile_;
21-
}
22-
23-
private:
24-
RuntimeSamplingProfile runtimeSamplingProfile_;
16+
RuntimeSamplingProfile runtimeSamplingProfile;
2517
};
2618

2719
} // namespace facebook::react::jsinspector_modern::tracing

0 commit comments

Comments
 (0)