From 4e77aebc17d407c2d503fc1b3fc128689dc6fb36 Mon Sep 17 00:00:00 2001 From: Nicolas Gallagher Date: Thu, 8 Apr 2021 17:19:05 -0700 Subject: [PATCH] Fix flow error in useEvent --- .../src/modules/useEvent/index.js | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/packages/react-native-web/src/modules/useEvent/index.js b/packages/react-native-web/src/modules/useEvent/index.js index 024dbd14..945b0d99 100644 --- a/packages/react-native-web/src/modules/useEvent/index.js +++ b/packages/react-native-web/src/modules/useEvent/index.js @@ -32,7 +32,7 @@ export default function useEvent( ): AddListener { const targetListeners = useStable(() => new Map()); - let addListener = useStable(() => { + const addListener = useStable(() => { const addEventListener = createEventHandle(event, options); return (target: EventTarget, callback: Callback) => { const removeTargetListener = targetListeners.get(target); @@ -50,16 +50,12 @@ export default function useEvent( useLayoutEffect(() => { return () => { - if (addListener != null) { - targetListeners.forEach((removeListener) => { - removeListener(); - }); - targetListeners.clear(); - } - addListener = null; + targetListeners.forEach((removeListener) => { + removeListener(); + }); + targetListeners.clear(); }; - }, [addListener]); + }, []); - // $FlowFixMe return addListener; }