Skip to content

M
mpd
Commit c9a9248c authored by Max Kellermann's avatar Max Kellermann Committed by Max Kellermann
Browse files
Options

java/Class: use Java::LocalObject

This eliminates all but one DeleteLocalRef() call.
parent 31f7fede
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 4 deletions
src/java/Class.hxx
View file @ c9a9248c
...@@ -58,23 +58,21 @@ public: ...@@ -58,23 +58,21 @@ public:
assert(env != nullptr); assert(env != nullptr);
assert(name != nullptr); assert(name != nullptr);
jclass cls = env->FindClass(name); const Java::Class cls{env, env->FindClass(name)};
assert(cls != nullptr); assert(cls != nullptr);
Set(env, cls); Set(env, cls);
env->DeleteLocalRef(cls);
} }
bool FindOptional(JNIEnv *env, const char *name) noexcept { bool FindOptional(JNIEnv *env, const char *name) noexcept {
assert(env != nullptr); assert(env != nullptr);
assert(name != nullptr); assert(name != nullptr);
jclass cls = env->FindClass(name); const Java::Class cls{env, env->FindClass(name)};
if (DiscardException(env)) if (DiscardException(env))
return false; return false;
Set(env, cls); Set(env, cls);
env->DeleteLocalRef(cls);
return true; return true;
} }
}; };
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment