Migrate YogaConfigJNIBase to Kotlin

Reviewed By: VladimirMakaev

Differential Revision: D80576929

fbshipit-source-id: 1310f77f5d9d489b780b14875454ebda7f7adfc9

java/build.gradle.kts

@@ -83,7 +83,8 @@ publishing {
       afterEvaluate { from(components["default"]) }
       pom {
         description.set(
-            "An embeddable and performant flexbox layout engine with bindings for multiple languages")
+            "An embeddable and performant flexbox layout engine with bindings for multiple languages"
+        )
         name.set(project.name)
         url.set("https://github.com/facebook/yoga.git")
         licenses {

java/com/facebook/yoga/YogaConfig.kt

@@ -10,7 +10,7 @@ package com.facebook.yoga
 public abstract class YogaConfig {
   public abstract fun setExperimentalFeatureEnabled(
       feature: YogaExperimentalFeature,
-      enabled: Boolean
+      enabled: Boolean,
   )
 
   public abstract fun setUseWebDefaults(useWebDefaults: Boolean)

java/com/facebook/yoga/YogaConfigJNIBase.java

@@ -1,62 +0,0 @@
-/*
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-package com.facebook.yoga;
-
-public abstract class YogaConfigJNIBase extends YogaConfig {
-
-  long mNativePointer;
-  private YogaLogger mLogger;
-
-  private YogaConfigJNIBase(long nativePointer) {
-    if (nativePointer == 0) {
-      throw new IllegalStateException("Failed to allocate native memory");
-    }
-    mNativePointer = nativePointer;
-  }
-
-  YogaConfigJNIBase() {
-    this(YogaNative.jni_YGConfigNewJNI());
-  }
-
-  YogaConfigJNIBase(boolean useVanillaJNI) {
-    this(YogaNative.jni_YGConfigNewJNI());
-  }
-
-  public void setExperimentalFeatureEnabled(YogaExperimentalFeature feature, boolean enabled) {
-    YogaNative.jni_YGConfigSetExperimentalFeatureEnabledJNI(mNativePointer, feature.intValue(), enabled);
-  }
-
-  public void setUseWebDefaults(boolean useWebDefaults) {
-    YogaNative.jni_YGConfigSetUseWebDefaultsJNI(mNativePointer, useWebDefaults);
-  }
-
-  public void setPointScaleFactor(float pixelsInPoint) {
-    YogaNative.jni_YGConfigSetPointScaleFactorJNI(mNativePointer, pixelsInPoint);
-  }
-
-  public void setErrata(YogaErrata errata) {
-    YogaNative.jni_YGConfigSetErrataJNI(mNativePointer, errata.intValue());
-  }
-
-  public YogaErrata getErrata() {
-    return YogaErrata.fromInt(YogaNative.jni_YGConfigGetErrataJNI(mNativePointer));
-  }
-
-  public void setLogger(YogaLogger logger) {
-    mLogger = logger;
-    YogaNative.jni_YGConfigSetLoggerJNI(mNativePointer, logger);
-  }
-
-  public YogaLogger getLogger() {
-    return mLogger;
-  }
-
-  protected long getNativePointer() {
-    return mNativePointer;
-  }
-}

java/com/facebook/yoga/YogaConfigJNIBase.kt

@@ -0,0 +1,61 @@
+/*
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+package com.facebook.yoga
+
+import com.facebook.yoga.YogaNative.jni_YGConfigGetErrataJNI
+import com.facebook.yoga.YogaNative.jni_YGConfigNewJNI
+import com.facebook.yoga.YogaNative.jni_YGConfigSetErrataJNI
+import com.facebook.yoga.YogaNative.jni_YGConfigSetExperimentalFeatureEnabledJNI
+import com.facebook.yoga.YogaNative.jni_YGConfigSetLoggerJNI
+import com.facebook.yoga.YogaNative.jni_YGConfigSetPointScaleFactorJNI
+import com.facebook.yoga.YogaNative.jni_YGConfigSetUseWebDefaultsJNI
+
+public abstract class YogaConfigJNIBase
+private constructor(@JvmField protected var nativePointer: Long) : YogaConfig() {
+  private var _logger: YogaLogger? = null
+
+  init {
+    check(nativePointer != 0L) { "Failed to allocate native memory" }
+  }
+
+  internal constructor() : this(jni_YGConfigNewJNI())
+
+  internal constructor(useVanillaJNI: Boolean) : this(jni_YGConfigNewJNI())
+
+  public override fun setExperimentalFeatureEnabled(
+      feature: YogaExperimentalFeature,
+      enabled: Boolean
+  ) {
+    YogaNative.jni_YGConfigSetExperimentalFeatureEnabledJNI(
+        nativePointer, feature.intValue(), enabled)
+  }
+
+  public override fun setUseWebDefaults(useWebDefaults: Boolean) {
+    YogaNative.jni_YGConfigSetUseWebDefaultsJNI(nativePointer, useWebDefaults)
+  }
+
+  public override fun setPointScaleFactor(pixelsInPoint: Float) {
+    YogaNative.jni_YGConfigSetPointScaleFactorJNI(nativePointer, pixelsInPoint)
+  }
+
+  public override fun setErrata(errata: YogaErrata) {
+    YogaNative.jni_YGConfigSetErrataJNI(nativePointer, errata.intValue())
+  }
+
+  public override fun getErrata(): YogaErrata =
+      YogaErrata.fromInt(YogaNative.jni_YGConfigGetErrataJNI(nativePointer))
+
+  public override fun setLogger(logger: YogaLogger) {
+    _logger = logger
+    YogaNative.jni_YGConfigSetLoggerJNI(nativePointer, logger)
+  }
+
+  public override fun getLogger(): YogaLogger = checkNotNull(_logger) { "YogaLogger is not set" }
+
+  public override fun getNativePointer(): Long = nativePointer
+}

java/com/facebook/yoga/YogaConfigJNIFinalizer.java

@@ -22,10 +22,10 @@ public class YogaConfigJNIFinalizer extends YogaConfigJNIBase {
    }
 
   public void freeNatives() {
-    if (mNativePointer != 0) {
+    if (nativePointer != 0) {
-      long nativePointer = mNativePointer;
+      long pointer = nativePointer;
-      mNativePointer = 0;
+      nativePointer = 0;
-      YogaNative.jni_YGConfigFreeJNI(nativePointer);
+      YogaNative.jni_YGConfigFreeJNI(pointer);
     }
   }
 }

java/com/facebook/yoga/YogaNative.kt

@@ -26,7 +26,7 @@ public object YogaNative {
   public external fun jni_YGConfigSetExperimentalFeatureEnabledJNI(
       nativePointer: Long,
       feature: Int,
-      enabled: Boolean
+      enabled: Boolean,
   )
 
   @JvmStatic
@@ -59,7 +59,7 @@ public object YogaNative {
   @JvmStatic
   public external fun jni_YGNodeSetIsReferenceBaselineJNI(
       nativePointer: Long,
-      isReferenceBaseline: Boolean
+      isReferenceBaseline: Boolean,
   )
 
   @JvmStatic public external fun jni_YGNodeIsReferenceBaselineJNI(nativePointer: Long): Boolean
@@ -74,7 +74,7 @@ public object YogaNative {
       width: Float,
       height: Float,
       nativePointers: LongArray,
-      nodes: Array<YogaNodeJNIBase>
+      nodes: Array<YogaNodeJNIBase>,
   )
 
   @JvmStatic public external fun jni_YGNodeMarkDirtyJNI(nativePointer: Long)
@@ -171,7 +171,7 @@ public object YogaNative {
   public external fun jni_YGNodeStyleSetMarginPercentJNI(
       nativePointer: Long,
       edge: Int,
-      percent: Float
+      percent: Float,
   )
 
   @JvmStatic public external fun jni_YGNodeStyleSetMarginAutoJNI(nativePointer: Long, edge: Int)
@@ -185,7 +185,7 @@ public object YogaNative {
   public external fun jni_YGNodeStyleSetPaddingPercentJNI(
       nativePointer: Long,
       edge: Int,
-      percent: Float
+      percent: Float,
   )
 
   @JvmStatic public external fun jni_YGNodeStyleGetBorderJNI(nativePointer: Long, edge: Int): Float
@@ -202,7 +202,7 @@ public object YogaNative {
   public external fun jni_YGNodeStyleSetPositionPercentJNI(
       nativePointer: Long,
       edge: Int,
-      percent: Float
+      percent: Float,
   )
 
   @JvmStatic public external fun jni_YGNodeStyleSetPositionAutoJNI(nativePointer: Long, edge: Int)
@@ -305,7 +305,7 @@ public object YogaNative {
   public external fun jni_YGNodeStyleSetGapPercentJNI(
       nativePointer: Long,
       gutter: Int,
-      gapLength: Float
+      gapLength: Float,
   )
 
   @JvmStatic
@@ -318,7 +318,7 @@ public object YogaNative {
   public external fun jni_YGNodeSetStyleInputsJNI(
       nativePointer: Long,
       styleInputsArray: FloatArray,
-      size: Int
+      size: Int,
   )
 
   @JvmStatic public external fun jni_YGNodeCloneJNI(nativePointer: Long): Long
@@ -326,6 +326,6 @@ public object YogaNative {
   @JvmStatic
   public external fun jni_YGNodeSetAlwaysFormsContainingBlockJNI(
       nativePointer: Long,
-      alwaysFormContainingBlock: Boolean
+      alwaysFormContainingBlock: Boolean,
   )
 }

java/com/facebook/yoga/YogaNodeJNIBase.java

@@ -57,7 +57,7 @@ public abstract class YogaNodeJNIBase extends YogaNode implements Cloneable {
   }
 
   YogaNodeJNIBase(YogaConfig config) {
-    this(YogaNative.jni_YGNodeNewWithConfigJNI(((YogaConfigJNIBase) config).mNativePointer));
+    this(YogaNative.jni_YGNodeNewWithConfigJNI(((YogaConfigJNIBase) config).nativePointer));
     mConfig = config; // makes sure the YogaConfig is not garbage collected
   }