Skip to content

Commit eb79ce3

Browse files
authored
Drop this comment for now.
It's historic, not related to the code as-is. That can be represented in the git history.
1 parent b58f4c7 commit eb79ce3

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

src/components/connectAdvanced.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -399,8 +399,6 @@ export default function connectAdvanced(
399399
return childPropsSelector(store.getState(), wrapperProps)
400400
}, [store, previousStateUpdateResult, wrapperProps])
401401

402-
// To avoid memory leak issue of function closure in useEffect, move useEffect functions out of component scope.
403-
404402
// We need this to execute synchronously every time we re-render. However, React warns
405403
// about useLayoutEffect in SSR, so we try to detect environment and fall back to
406404
// just useEffect instead to avoid the warning, since neither will run anyway.

0 commit comments

Comments
 (0)