Fix castTo and publish 0.10.0 (#1323)

Author: HosseinYousefi

pkgs/jni/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.10.0
+
+- Fixed an issue with `JObject.castTo` where the type checking could fail in
+  debug mode.
+- Used `package:dart_flutter_team_lints`.
+
 ## 0.9.3
 
 - Added lifetime event handling for the thread-local JNI env. Now

pkgs/jni/lib/src/jobject.dart

@@ -43,7 +43,7 @@ final class JObjectType extends JObjType<JObject> {
 /// A high-level wrapper for JNI global object reference.
 ///
 /// This is the base class for classes generated by `jnigen`.
-class JObject {
+class JObject implements Finalizable {
   final JReference reference;
 
   late final JObjType<JObject> $type = type;
@@ -83,7 +83,12 @@ class JObject {
     bool releaseOriginal = false,
   }) {
     assert(
-      Jni.env.IsInstanceOf(reference.pointer, type.jClass.reference.pointer),
+      () {
+        final jClass = type.jClass.reference.toPointer();
+        final canBeCasted = Jni.env.IsInstanceOf(reference.pointer, jClass);
+        Jni.env.DeleteGlobalRef(jClass);
+        return canBeCasted;
+      }(),
       'The object must be of type "${type.signature}".',
     );
     if (releaseOriginal) {

pkgs/jni/lib/src/jreference.dart

@@ -39,7 +39,7 @@ final class _JFinalizable implements Finalizable {
 }
 
 @pragma('vm:deeply-immutable')
-abstract final class JReference {
+abstract final class JReference implements Finalizable {
   final _JFinalizable _finalizable;
 
   JReference(this._finalizable);

pkgs/jni/test/exception_test.dart

@@ -30,7 +30,7 @@ void main() {
     if (!caught) {
       stderr.writeln('Expected HelperNotFoundException\n'
           'Read exception_test.dart for details.');
-      exit(-1);
+      exit(1);
     }
   }
   run(testRunner: test);

pkgs/jni/tool/generate_ide_files.dart

@@ -22,7 +22,7 @@ void runCommand(String exec, List<String> args, String workingDirectory) {
     stdout.writeln(process.stdout);
     stderr.writeln(process.stderr);
     stderr.writeln('command failed with exit code ${process.exitCode}');
-    exit(-1);
+    exit(1);
   }
 }
 

pkgs/jni/tool/wrapper_generators/generate_c_extensions.dart

@@ -179,7 +179,7 @@ ResultWrapper getResultWrapper(String returnType) {
         return ResultWrapper.forJValueField(jfields[returnType]!);
       }
       stderr.writeln('Unknown type $returnType for return type');
-      exit(-1);
+      exit(1);
   }
 }
 

pkgs/jni/tool/wrapper_generators/generate_dart_extensions.dart

@@ -71,7 +71,7 @@ String getCheckedGetter(Type returnType) {
     return mappings[cType]!;
   }
   stderr.writeln('Unknown return type: $cType');
-  exit(-1);
+  exit(1);
 }
 
 String? getGlobalEnvExtensionFunction(Member field, Type? checkedReturnType) {

pkgs/jnigen/CHANGELOG.md

@@ -1,7 +1,7 @@
-## 0.10.0-wip
+## 0.10.0
 
 - Added support for Kotlin's top-level functions and fields.
-- Use `package:dart_flutter_team_lints`.
+- Used `package:dart_flutter_team_lints`.
 
 ## 0.9.3