From 499d28d021a5330e9d5d319cfc5b8fbbc91300de Mon Sep 17 00:00:00 2001
From: Amir Shalem <amirshalem@fb.com>
Date: Tue, 17 Sep 2019 06:52:49 -0700
Subject: [PATCH] Expose native pointer thru an interface function for
 YogaConfig

Summary:
Expose native pointer thru an interface function for YogaConfig (its package private to `com.facebook.yoga` namespace),
This way we can make later on YogaConfig a pure abstract class.

Plus, it makes sure external users don't modify the pointer

Reviewed By: SidharthGuglani

Differential Revision: D17266401

fbshipit-source-id: f39b488cea0b73bc3578bb3aa90ab00139bf9271
---
 java/com/facebook/yoga/YogaConfig.java      | 6 +++++-
 java/com/facebook/yoga/YogaNodeJNIBase.java | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/java/com/facebook/yoga/YogaConfig.java b/java/com/facebook/yoga/YogaConfig.java
index bc2538da..49a4c44b 100644
--- a/java/com/facebook/yoga/YogaConfig.java
+++ b/java/com/facebook/yoga/YogaConfig.java
@@ -12,7 +12,7 @@ public class YogaConfig {
 
   public static int SPACING_TYPE = 1;
 
-  long mNativePointer;
+  private long mNativePointer;
   private YogaLogger mLogger;
   private YogaNodeCloneFunction mYogaNodeCloneFunction;
 
@@ -76,4 +76,8 @@ public class YogaConfig {
   public YogaLogger getLogger() {
     return mLogger;
   }
+
+  long getNativePointer() {
+    return mNativePointer;
+  }
 }
diff --git a/java/com/facebook/yoga/YogaNodeJNIBase.java b/java/com/facebook/yoga/YogaNodeJNIBase.java
index a2f7845e..f253b95c 100644
--- a/java/com/facebook/yoga/YogaNodeJNIBase.java
+++ b/java/com/facebook/yoga/YogaNodeJNIBase.java
@@ -58,7 +58,7 @@ public abstract class YogaNodeJNIBase extends YogaNode implements Cloneable {
   }
 
   YogaNodeJNIBase(YogaConfig config) {
-    this(YogaNative.jni_YGNodeNewWithConfig(config.mNativePointer));
+    this(YogaNative.jni_YGNodeNewWithConfig(config.getNativePointer()));
   }
 
   public void reset() {