Fixes #38 and another oversight

samolego · web-flow · commit 4a29f8e7f188 · 2024-01-07T20:44:53.000+01:00
Fixes deleting local reference where it isn't created.
Fixes setting a boolean field with `SetLongField` instead of `SetBooleanField`
diff --git a/src/main/cpp/jllama.cpp b/src/main/cpp/jllama.cpp
@@ -361,7 +361,6 @@ static void jllama_log_callback(enum ggml_log_level level, const char *text, voi
 
     env->CallVoidMethod(g_log_callback, m_biconsumer_accept, java_log_level, java_text);
 
-    env->DeleteLocalRef(java_log_level);
     env->DeleteLocalRef(java_text);
 }
 
@@ -1251,7 +1250,7 @@ JNIEXPORT jobject JNICALL Java_de_kherud_llama_LlamaModel_getNext(JNIEnv *env, j
 
     if (!llama->has_next_token)
     {
-        env->SetLongField(iter, f_iter_has_next, false);
+        env->SetBooleanField(iter, f_iter_has_next, false);
         // llama.mutex.unlock();
         // lock.release();
     }

Original file line number	Diff line number	Diff line change
`@@ -361,7 +361,6 @@ static void jllama_log_callback(enum ggml_log_level level, const char *text, voi`
`361`	`361`
`362`	`362`	`env->CallVoidMethod(g_log_callback, m_biconsumer_accept, java_log_level, java_text);`
`363`	`363`
`364`		`- env->DeleteLocalRef(java_log_level);`
`365`	`364`	`env->DeleteLocalRef(java_text);`
`366`	`365`	`}`
`367`	`366`
`@@ -1251,7 +1250,7 @@ JNIEXPORT jobject JNICALL Java_de_kherud_llama_LlamaModel_getNext(JNIEnv *env, j`
`1251`	`1250`
`1252`	`1251`	`if (!llama->has_next_token)`
`1253`	`1252`	`{`
`1254`		`- env->SetLongField(iter, f_iter_has_next, false);`
	`1253`	`+ env->SetBooleanField(iter, f_iter_has_next, false);`
`1255`	`1254`	`// llama.mutex.unlock();`
`1256`	`1255`	`// lock.release();`
`1257`	`1256`	`}`