From 7e9e6c1ce59b17ddf0d483c61eb857ce1dfc0dee Mon Sep 17 00:00:00 2001 From: Sidharth Guglani Date: Wed, 13 Mar 2019 06:21:58 -0700 Subject: [PATCH] moved together the java object fields which are set always Summary: Moved all layout outputs fields which are set always on yoga node java object. This change set is for adding experiment for layout outputs batching using a float array Reviewed By: davidaurelio Differential Revision: D14355025 fbshipit-source-id: 371d9c49fcb631efa8a79b62f8051ba5a17c232c --- ReactAndroid/src/main/jni/first-party/yogajni/jni/YGJNI.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ReactAndroid/src/main/jni/first-party/yogajni/jni/YGJNI.cpp b/ReactAndroid/src/main/jni/first-party/yogajni/jni/YGJNI.cpp index 8463421fb..0cd97be88 100644 --- a/ReactAndroid/src/main/jni/first-party/yogajni/jni/YGJNI.cpp +++ b/ReactAndroid/src/main/jni/first-party/yogajni/jni/YGJNI.cpp @@ -175,6 +175,8 @@ static void YGTransferLayoutOutputsRecursive( obj->setFieldValue( doesLegacyStretchBehaviour, YGNodeLayoutGetDidLegacyStretchFlagAffectLayout(root)); + obj->setFieldValue(hasNewLayoutField, true); + YGTransferLayoutDirection(root, obj); if ((edgeSetFlag & MARGIN) == MARGIN) { obj->setFieldValue( @@ -207,8 +209,6 @@ static void YGTransferLayoutOutputsRecursive( borderBottomField, YGNodeLayoutGetBorder(root, YGEdgeBottom)); } - obj->setFieldValue(hasNewLayoutField, true); - YGTransferLayoutDirection(root, obj); root->setHasNewLayout(false); for (uint32_t i = 0; i < YGNodeGetChildCount(root); i++) {