Comments only. Clarify a few Venice and SVC methods

Summary: Changelog: [Internal] Reviewed By: fkgozali Differential Revision: D33468276 fbshipit-source-id: bcc62ba1e728bba2492dbbbb0e4569285effeca7
facebook · Jan 8, 2022 · 6be46b0 · 6be46b0
1 parent 9daae5a
commit 6be46b0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/Libraries/NativeComponent/NativeComponentRegistry.js b/Libraries/NativeComponent/NativeComponentRegistry.js
@@ -116,6 +116,7 @@ export function getWithFallback_DEPRECATED<Config>(
   viewConfigProvider: () => PartialViewConfig,
 ): React.AbstractComponent<Config> {
   if (getRuntimeConfig == null) {
+    // `getRuntimeConfig == null` when static view configs are disabled
     // If `setRuntimeConfigProvider` is not configured, use native reflection.
     if (hasNativeViewConfig(name)) {
       return get<Config>(name, viewConfigProvider);

diff --git a/React/Fabric/Mounting/RCTComponentViewFactory.mm b/React/Fabric/Mounting/RCTComponentViewFactory.mm
@@ -41,6 +41,7 @@
 
 using namespace facebook::react;
 
+// Allow JS runtime to register native components as needed. For static view configs.
 void RCTInstallNativeComponentRegistryBinding(facebook::jsi::Runtime &runtime)
 {
   auto hasComponentProvider = [](std::string const &name) -> bool {