Fabric: Remove redundant class forward declaration

Summary: Remove redundant forward declaration and make default constructor explicit. changelog: [internal] Reviewed By: shergin Differential Revision: D18032511 fbshipit-source-id: 18d3f9c869a69949e6965cd7e32e0f284723f2e3
facebook · Oct 21, 2019 · 7e49cf1 · 7e49cf1
1 parent fd07fc9
commit 7e49cf1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/ReactCommon/fabric/core/state/StateCoordinator.h b/ReactCommon/fabric/core/state/StateCoordinator.h
@@ -16,8 +16,6 @@
 namespace facebook {
 namespace react {
 
-class ShadowNode;
-
 /*
  * Coordinates a vision of the same state values between shadow nodes from
  * the same family.

diff --git a/ReactCommon/fabric/core/state/StateTarget.h b/ReactCommon/fabric/core/state/StateTarget.h
@@ -24,12 +24,12 @@ class StateTarget {
   /*
    * Creates an empty target.
    */
-  StateTarget();
+  explicit StateTarget();
 
   /*
    * Creates a target which points to a given `ShadowNode`.
    */
-  StateTarget(const ShadowNode &shadowNode);
+  explicit StateTarget(const ShadowNode &shadowNode);
 
   /*
    * Copyable and moveable.