From 19bca222dc3cdeb58d5dc75e868ff001ac66ca82 Mon Sep 17 00:00:00 2001 From: Samuel Susla Date: Mon, 22 Nov 2021 04:20:07 -0800 Subject: [PATCH] Remove asserts in EventTarget Summary: changelog: [internal] Reviewed By: philIip Differential Revision: D32393073 fbshipit-source-id: 9e228000291d67f3a0cedaa152c0376e11d7dcf6 --- ReactCommon/react/renderer/core/EventTarget.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ReactCommon/react/renderer/core/EventTarget.cpp b/ReactCommon/react/renderer/core/EventTarget.cpp index b8d84547016..c43dd565474 100644 --- a/ReactCommon/react/renderer/core/EventTarget.cpp +++ b/ReactCommon/react/renderer/core/EventTarget.cpp @@ -40,8 +40,10 @@ void EventTarget::retain(jsi::Runtime &runtime) const { // particular implementation of JSI was able to detect this inconsistency and // dealt with it, but some JSI implementation may not support this feature and // that case will lead to a crash in those environments. - react_native_assert(!strongInstanceHandle_.isNull()); - react_native_assert(!strongInstanceHandle_.isUndefined()); + + // TODO: Replace with mustfix once mustfix is ready in React Native. + // react_native_assert(!strongInstanceHandle_.isNull()); + // react_native_assert(!strongInstanceHandle_.isUndefined()); } void EventTarget::release(jsi::Runtime &runtime) const {