Revert "🦄 refactor(Android): Add support to Optional, Stream, `Zo…

…nedDateTime` for Android"

This reverts commit 8d47fb6.

android/javet-android/build.gradle.kts

@@ -1,5 +1,5 @@
 /*
- *    Copyright 2021-2022. caoccao.com Sam Cao
+ *    Copyright 2021. caoccao.com Sam Cao
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -55,9 +55,8 @@ android {
 }
 
 dependencies {
-    // https://developer.android.com/studio/write/java8-support
-    coreLibraryDesugaring("com.android.tools:desugar_jdk_libs:1.1.5")
-    implementation("androidx.appcompat:appcompat:1.4.1")
+    implementation("androidx.appcompat:appcompat:1.3.1")
+    implementation("org.threeten:threetenbp:1.5.1")
     testImplementation("junit:junit:4.13.2")
     androidTestImplementation("androidx.test.ext:junit:1.1.3")
     androidTestImplementation("androidx.test.espresso:espresso-core:3.4.0")

docs/release_notes/release_notes_1_1.html

@@ -295,7 +295,6 @@ <h1>Release Notes 1.1.x<a class="headerlink" href="#release-notes-1-1-x" title="
 <h2>1.1.1<a class="headerlink" href="#id1" title="Permalink to this headline">¶</a></h2>
 <ul class="simple">
 <li><p>Enhanced createV8ValueArrayBuffer with ByteBuffer</p></li>
-<li><p>Added support to <cite>Optional</cite>, <cite>Stream</cite>, <cite>ZonedDateTime</cite> for Android</p></li>
 </ul>
 </div>
 <div class="section" id="v8-v9-8">

docs/release_notes/release_notes_1_1.rst

@@ -6,7 +6,6 @@ Release Notes 1.1.x
 -------------
 
 * Enhanced createV8ValueArrayBuffer with ByteBuffer
-* Added support to `Optional`, `Stream`, `ZonedDateTime` for Android
 
 1.1.0 V8 v9.8
 -------------

docs/tutorial/basic/installation.html

@@ -455,6 +455,8 @@ <h3>Android<a class="headerlink" href="#android" title="Permalink to this headli
 <ul class="simple">
 <li><p>Only V8 mode is supported for Android. Supporting Node.js mode implies huge amount of work, but is not mission impossible. Please contact the maintainer for detail.</p></li>
 <li><p>If you need Node.js features on Android, please refer to project <a class="reference external" href="https://github.com/caoccao/Javenode">Javenode</a>.</p></li>
+<li><p><a class="reference external" href="https://developer.android.com/reference/java/time/ZonedDateTime">ZonedDateTime</a> (since ABI 26) is polyfilled by <code class="docutils literal notranslate"><span class="pre">org.threeten.bp.ZonedDateTime</span></code> so that Javet can work well with low ABI version.</p></li>
+<li><p><cite>Stream</cite> and <cite>Optional</cite> are not supported because they are not available with ABI version 23-24.</p></li>
 <li><p>If you need ABI version 21/22, please use Javet v1.0.x or contact the maintainer for details.</p></li>
 </ul>
 </div>

docs/tutorial/basic/installation.rst

@@ -104,4 +104,6 @@ Android x86_64              Yes (ABI >= 23)
 
     * Only V8 mode is supported for Android. Supporting Node.js mode implies huge amount of work, but is not mission impossible. Please contact the maintainer for detail.
     * If you need Node.js features on Android, please refer to project `Javenode <https://github.com/caoccao/Javenode>`_.
+    * `ZonedDateTime <https://developer.android.com/reference/java/time/ZonedDateTime>`_ (since ABI 26) is polyfilled by ``org.threeten.bp.ZonedDateTime`` so that Javet can work well with low ABI version.
+    * `Stream` and `Optional` are not supported because they are not available with ABI version 23-24.
     * If you need ABI version 21/22, please use Javet v1.0.x or contact the maintainer for details.

scripts/python/patch_android_build.py

@@ -30,9 +30,19 @@ class PatchAndroidBuild(object):
   def __init__(self) -> None:
     self._line_separator = '\n'
     self._patch_dict = {
+      'java.time.': 'org.threeten.bp.',
       '.getParameterCount()': '.getParameterTypes().length',
     }
     self._comment_list = [
+      'import java.util.Optional;',
+      'import java.util.OptionalDouble;',
+      'import java.util.OptionalInt;',
+      'import java.util.OptionalLong;',
+      'import java.util.stream.BaseStream;',
+      'import java.util.stream.DoubleStream;',
+      'import java.util.stream.IntStream;',
+      'import java.util.stream.LongStream;',
+      'import java.util.stream.Stream;',
       'import javax.management.ListenerNotFoundException;',
       'import javax.management.Notification;',
       'import javax.management.NotificationEmitter;',

src/main/java/com/caoccao/javet/interop/callback/V8FunctionCallback.java

@@ -75,6 +75,7 @@ private static Object convert(IJavetConverter converter, Class<?> expectedClass,
                 } else if (expectedClass == char.class) {
                     return config.getDefaultChar();
                 }
+                /* if not defined ANDROID */
             } else if (expectedClass == Optional.class) {
                 return Optional.empty();
             } else if (expectedClass == OptionalInt.class) {
@@ -91,6 +92,7 @@ private static Object convert(IJavetConverter converter, Class<?> expectedClass,
                 return DoubleStream.empty();
             } else if (expectedClass == LongStream.class) {
                 return LongStream.empty();
+                /* end if */
             }
         } else if (expectedClass.isAssignableFrom(v8Value.getClass())) {
             // Skip assignable
@@ -257,6 +259,7 @@ private static Object convert(IJavetConverter converter, Class<?> expectedClass,
                             return convertedString.length() > 0 ?
                                     convertedString.charAt(0) : converter.getConfig().getDefaultChar();
                         }
+                        /* if not defined ANDROID */
                     } else if (expectedClass == Optional.class) {
                         return Optional.of(convertedObject);
                     } else if (expectedClass == OptionalInt.class) {
@@ -291,6 +294,7 @@ private static Object convert(IJavetConverter converter, Class<?> expectedClass,
                         if (doubleStream != null) {
                             return doubleStream;
                         }
+                        /* end if */
                     }
                 }
             } catch (Throwable ignored) {

src/main/java/com/caoccao/javet/interop/converters/JavetObjectConverter.java

@@ -431,6 +431,7 @@ protected <T extends V8Value> T toV8Value(
                 v8Value = v8ValueArray;
                 v8Scope.setEscapable();
             }
+            /* if not defined ANDROID */
         } else if (object instanceof BaseStream) {
             try (V8Scope v8Scope = v8Runtime.getV8Scope()) {
                 V8ValueArray v8ValueArray = v8Scope.createV8ValueArray();
@@ -443,6 +444,7 @@ protected <T extends V8Value> T toV8Value(
                 v8Value = v8ValueArray;
                 v8Scope.setEscapable();
             }
+            /* end if */
         } else if (object instanceof IJavetEntityFunction) {
             final IJavetEntityFunction javetEntityFunction = (IJavetEntityFunction) object;
             String sourceCode = javetEntityFunction.getJSFunctionType().isUserDefined() ?

src/main/java/com/caoccao/javet/interop/converters/JavetPrimitiveConverter.java

@@ -113,6 +113,7 @@ protected <T extends V8Value> T toV8Value(
             v8Value = v8Runtime.createV8ValueInteger((Byte) object);
         } else if (object instanceof Character) {
             v8Value = v8Runtime.createV8ValueString(((Character) object).toString());
+            /* if not defined ANDROID */
         } else if (object instanceof Optional) {
             Optional<?> optional = (Optional<?>) object;
             v8Value = optional.isPresent() ? toV8Value(v8Runtime, optional.get(), depth) : v8Runtime.createV8ValueNull();
@@ -125,6 +126,7 @@ protected <T extends V8Value> T toV8Value(
         } else if (object instanceof OptionalLong) {
             OptionalLong optional = (OptionalLong) object;
             v8Value = optional.isPresent() ? toV8Value(v8Runtime, optional.getAsLong(), depth) : v8Runtime.createV8ValueNull();
+            /* end if */
         } else {
             v8Value = v8Runtime.createV8ValueUndefined();
         }

src/main/java/com/caoccao/javet/utils/JavetReflectionUtils.java

@@ -89,13 +89,15 @@ public static String getMethodNameFromLambda(Serializable lambda) {
      * @return the method name set
      * @since 0.9.13
      */
+    /* if not defined ANDROID */
     public static Set<String> getMethodNameSetFromLambdas(Serializable... lambdas) {
         return Stream.of(Objects.requireNonNull(lambdas))
                 .filter(Objects::nonNull)
                 .map(JavetReflectionUtils::getMethodNameFromLambda)
                 .filter(Objects::nonNull)
                 .collect(Collectors.toSet());
     }
+    /* end if */
 
     /**
      * Safe set accessible.

src/main/java/com/caoccao/javet/utils/JavetTypeUtils.java

@@ -138,6 +138,7 @@ public static Object toApproximatePrimitive(Class<?> expectedClass, Object objec
      * @return the double stream
      * @since 0.9.13
      */
+    /* if not defined ANDROID */
     public static DoubleStream toDoubleStream(Object object) {
         if (object instanceof double[]) {
             return DoubleStream.of((double[]) object);
@@ -149,6 +150,7 @@ public static DoubleStream toDoubleStream(Object object) {
         }
         return null;
     }
+    /* end if */
 
     /**
      * To exact primitive object.
@@ -193,6 +195,7 @@ public static Object toExactPrimitive(Class<?> expectedClass, Object object) {
      * @return the int stream
      * @since 0.9.13
      */
+    /* if not defined ANDROID */
     public static IntStream toIntStream(Object object) {
         if (object instanceof int[]) {
             return IntStream.of((int[]) object);
@@ -204,6 +207,7 @@ public static IntStream toIntStream(Object object) {
         }
         return null;
     }
+    /* end if */
 
     /**
      * Convert object to long stream.
@@ -212,6 +216,7 @@ public static IntStream toIntStream(Object object) {
      * @return the long stream
      * @since 0.9.13
      */
+    /* if not defined ANDROID */
     public static LongStream toLongStream(Object object) {
         if (object instanceof long[]) {
             return LongStream.of((long[]) object);
@@ -223,6 +228,7 @@ public static LongStream toLongStream(Object object) {
         }
         return null;
     }
+    /* end if */
 
     /**
      * Convert object to stream.
@@ -231,6 +237,7 @@ public static LongStream toLongStream(Object object) {
      * @return the stream
      * @since 0.9.13
      */
+    /* if not defined ANDROID */
     public static Stream<?> toStream(Object object) {
         if (object.getClass().isArray()) {
             if (object instanceof int[]) {
@@ -282,4 +289,5 @@ public static Stream<?> toStream(Object object) {
         }
         return null;
     }
+    /* end if */
 }