Add issue references to TODOs

.github/workflows/test-package.yml

@@ -78,6 +78,7 @@ jobs:
 
   ## TODO: More minimal test on windows after fixing dev dependency.
   ## i.e do not rerun analyze and format steps, and do not require flutter.
+  ## IssueRef: https://github.com/dart-lang/jni_gen/issues/15
 
   test_jni:
     runs-on: ubuntu-latest

jni/bin/setup.dart

@@ -15,6 +15,7 @@ class CommandRunner {
   int? time;
   // TODO: time commands
   // TODO: Run all commands in single shell instance
+  // IssueRef: https://github.com/dart-lang/jni_gen/issues/14
   Future<CommandRunner> run(
       String exec, List<String> args, String workingDir) async {
     if (printCmds) {

jni/example/android/app/build.gradle

@@ -43,7 +43,6 @@ android {
     }
 
     defaultConfig {
-        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
         applicationId "dev.dart.jni_example"
         // You can update the following values to match your application needs.
         // For more information, see: https://docs.flutter.dev/deployment/android#reviewing-the-build-configuration.

jni/lib/src/extensions.dart

@@ -60,9 +60,10 @@ extension AdditionalJniEnvMethods on Pointer<JniEnv> {
   void checkException({bool describe = false}) {
     final exc = ExceptionOccurred();
     if (exc != nullptr) {
-      // TODO: Doing this every time is expensive
-      // Should lookup and cache method reference
-      // and keep it alive by keeping a reference to Exception class
+      // TODO: Doing this every time is expensive.
+      // Should lookup and cache method reference,
+      // and keep it alive by keeping a reference to Exception class.
+      // IssueRef: https://github.com/dart-lang/jni_gen/issues/13
       final ecls = GetObjectClass(exc);
       final toStr = GetMethodID(ecls, _toString, _toStringSig);
       final jstr = CallObjectMethod(exc, toStr);

jni/lib/src/jni_class.dart

@@ -124,8 +124,6 @@ class JniClass {
   ///
   /// Useful in expression chains.
   T use<T>(T Function(JniClass) callback) {
-    // TODO: Maybe use a mixin or something?
-    // JniClass and JniObject have some similar functionality.
     _checkDeleted();
     final result = callback(this);
     delete();

jni/src/dartjni.c

@@ -34,6 +34,7 @@ void SetJNILogging(int level) {
 
 void jni_log(int level, const char *format, ...) {
 	// TODO: Not working
+	// IssueRef: https://github.com/dart-lang/jni_gen/issues/16
 	if (level >= jni_log_level) {
 		va_list args;
         va_start(args, format);