Skip to content
This repository was archived by the owner on Mar 8, 2020. It is now read-only.

Commit 55abd1f

Browse files
committed
address review feedback
Signed-off-by: Alexander Bezzubov <[email protected]>
1 parent 1a30b7c commit 55abd1f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/main/native/jni_utils.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ JNIEnv *getJNIEnv() {
1414

1515
case JNI_EDETACHED: // Thread is detached, need to attach
1616
jvm->AttachCurrentThread((void **)&pEnv, NULL);
17-
// TODO: this is a memory leak (at least)
17+
// FIXME(bzz): this is a memory leak (at least)
1818
// - jvm->DetachCurrentThread() is never called
1919
// - local references are _never_ deleted :scream:
2020
// https://developer.android.com/training/articles/perf-jni#local-and-global-references

0 commit comments

Comments
 (0)