Fix CQS signal performance-faster-string-find in xplat/js/react-native-github/packages/react-native/ReactCommon/react/runtime (#52568)

generatedunixname89002005287564 · facebook-github-bot · commit b8550251d86b · 2025-07-11T10:41:33.000-07:00
Summary: Pull Request resolved: #52568 Reviewed By: dtolnay Differential Revision: D78146248 fbshipit-source-id: 8b93b09d46832d6e67c1ded075eb99b9bdd1f0e2
diff --git a/packages/react-native/ReactCommon/react/runtime/ReactInstance.cpp b/packages/react-native/ReactCommon/react/runtime/ReactInstance.cpp
@@ -200,7 +200,7 @@ namespace {
 // Copied from JSIExecutor.cpp
 // basename_r isn't in all iOS SDKs, so use this simple version instead.
 std::string simpleBasename(const std::string& path) {
-  size_t pos = path.rfind("/");
+  size_t pos = path.rfind('/');
   return (pos != std::string::npos) ? path.substr(pos) : path;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -200,7 +200,7 @@ namespace {`
`200`	`200`	`// Copied from JSIExecutor.cpp`
`201`	`201`	`// basename_r isn't in all iOS SDKs, so use this simple version instead.`
`202`	`202`	`std::string simpleBasename(const std::string& path) {`
`203`		`- size_t pos = path.rfind("/");`
	`203`	`+ size_t pos = path.rfind('/');`
`204`	`204`	`return (pos != std::string::npos) ? path.substr(pos) : path;`
`205`	`205`	`}`
`206`	`206`