Skip to content

Commit 8686844

Browse files
coadofacebook-github-bot
authored andcommitted
Make StackTraceParser.h private in jserrorhandler
Summary: `StackTraceParser` is an implementation detail of `JsErrorHandler`: it is only used by `JsErrorHandler.cpp` and its own unit test, and `JsErrorHandler.h` does not include it. Mark it as a private header instead of a for-frameworks one. - Swap `FrameworksGuard.h` for `PrivateGuard.h` in `StackTraceParser.h`. - Include `JsErrorHandler.h` via its namespaced path so the header resolves when it is not exported alongside it. - Stop shipping `StackTraceParser.h` as a header for prebuilt `React-jserrorhandler` pods. Changelog: [Internal] Differential Revision: D122770464
1 parent c63744d commit 8686844

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

‎packages/react-native/ReactCommon/jserrorhandler/React-jserrorhandler.podspec‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ Pod::Spec.new do |s|
3434
s.platforms = min_supported_versions
3535
s.source = source
3636
s.header_dir = "jserrorhandler"
37-
s.source_files = podspec_sources(["ErrorUtils.{cpp,h}", "JsErrorHandler.{cpp,h}", "StackTraceParser.{cpp,h}"], ["ErrorUtils.h", "JsErrorHandler.h", "StackTraceParser.h"])
37+
s.source_files = podspec_sources(["ErrorUtils.{cpp,h}", "JsErrorHandler.{cpp,h}", "StackTraceParser.{cpp,h}"], ["ErrorUtils.h", "JsErrorHandler.h"])
3838
s.pod_target_xcconfig = {
3939
"HEADER_SEARCH_PATHS" => header_search_paths.join(' '),
4040
"USE_HEADERMAP" => "YES",

‎packages/react-native/ReactCommon/jserrorhandler/StackTraceParser.h‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@
77

88
#pragma once
99

10-
#include <react/cxxstableapi/FrameworksGuard.h>
10+
#include <react/cxxstableapi/PrivateGuard.h>
1111

12+
#include <jserrorhandler/JsErrorHandler.h>
1213
#include <string>
1314
#include <vector>
14-
#include "JsErrorHandler.h"
1515

1616
namespace facebook::react {
1717

0 commit comments

Comments
 (0)