From debc32b075427e5bf711ada8d375575c73f73458 Mon Sep 17 00:00:00 2001 From: Pieter De Baets Date: Wed, 25 May 2022 08:36:38 -0700 Subject: [PATCH] Fix startListeningToAnimatedNodeValue in NativeAnimated batch mode Summary: This op code was incorrectly configured to take two args, while it only takes one. Changelog: [Internal] Differential Revision: D36664590 fbshipit-source-id: 6e1fdb9f64bbd32fbe05bbd174f94ae57292bcf9 --- .../java/com/facebook/react/animated/NativeAnimatedModule.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/animated/NativeAnimatedModule.java b/ReactAndroid/src/main/java/com/facebook/react/animated/NativeAnimatedModule.java index af25f450616..275fcdea53f 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/animated/NativeAnimatedModule.java +++ b/ReactAndroid/src/main/java/com/facebook/react/animated/NativeAnimatedModule.java @@ -1047,6 +1047,7 @@ public class NativeAnimatedModule extends NativeAnimatedModuleSpec BatchExecutionOpCodes command = BatchExecutionOpCodes.fromId(opsAndArgs.getInt(i++)); switch (command) { case OP_CODE_GET_VALUE: + case OP_START_LISTENING_TO_ANIMATED_NODE_VALUE: case OP_STOP_LISTENING_TO_ANIMATED_NODE_VALUE: case OP_CODE_STOP_ANIMATION: case OP_CODE_FLATTEN_ANIMATED_NODE_OFFSET: @@ -1059,7 +1060,6 @@ public class NativeAnimatedModule extends NativeAnimatedModuleSpec break; case OP_CODE_CREATE_ANIMATED_NODE: case OP_CODE_UPDATE_ANIMATED_NODE_CONFIG: - case OP_START_LISTENING_TO_ANIMATED_NODE_VALUE: case OP_CODE_CONNECT_ANIMATED_NODES: case OP_CODE_DISCONNECT_ANIMATED_NODES: case OP_CODE_SET_ANIMATED_NODE_VALUE: @@ -1115,7 +1115,6 @@ public class NativeAnimatedModule extends NativeAnimatedModuleSpec break; case OP_START_LISTENING_TO_ANIMATED_NODE_VALUE: final int tag = opsAndArgs.getInt(i++); - final int value = opsAndArgs.getInt(i++); final AnimatedNodeValueListener listener = new AnimatedNodeValueListener() { public void onValueUpdate(double value) {