Facebook/react-native
1
0
Fork 0
mirror of https://github.com/facebook/react-native.git synced 2025-11-01 09:14:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

Animated: Omit allowlist in avoidStateUpdateInAnimatedPropsMemo Experiment (#50232)

Browse Source 
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/50232

While reviewing the `avoidStateUpdateInAnimatedPropsMemo` experiment, I noticed that the control and test groups were invoking `areCompositeKeysEqual` differently:

- Test group was passing in `allowlist`.
- Control group was not passing in `allowlist`.

Passing it in is technically more correct, but let's restore the control group behavior for now to isolate the tested changes.

Changelog:
[Internal]

Reviewed By: javache

Differential Revision: D71746745

fbshipit-source-id: aa34db0532ef53ead08bbd632aab0994b31b9340
This commit is contained in:
Tim Yung
2025-03-26 08:07:53 -07:00
committed by Facebook GitHub Bot
parent 4789744164
commit 2abe623d9f
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/react-native/src/private/animated/createAnimatedPropsMemoHook.js
Vendored
+1 -2
View File
@@ -91,8 +91,7 @@ export function createAnimatedPropsMemoHook(
const prev = prevRef.current;
const next =
prev != null &&
areCompositeKeysEqual(prev.compositeKey, compositeKey, allowlist)
prev != null && areCompositeKeysEqual(prev.compositeKey, compositeKey)
? prev
: {
compositeKey,