From 7e49cf1930bf01a6cd2491d3d081099685e1f4f2 Mon Sep 17 00:00:00 2001 From: Samuel Susla Date: Mon, 21 Oct 2019 06:22:14 -0700 Subject: [PATCH] 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 --- ReactCommon/fabric/core/state/StateCoordinator.h | 2 -- ReactCommon/fabric/core/state/StateTarget.h | 4 ++-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/ReactCommon/fabric/core/state/StateCoordinator.h b/ReactCommon/fabric/core/state/StateCoordinator.h index e1d652ef064d88..8fbf7b959bc1bb 100644 --- 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 index 5ea5aedd712d06..de847e8ae64de5 100644 --- 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.