Skip to content

Commit

Permalink
remove redundant write_return_value function
Browse files Browse the repository at this point in the history
  • Loading branch information
CedNaru committed Oct 14, 2023
1 parent b43b556 commit 474a8a6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
6 changes: 3 additions & 3 deletions src/memory/transfer_context.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ uint32_t TransferContext::read_args(jni::Env& p_env, Variant* args) {
}

void TransferContext::write_return_value(jni::Env& p_env, Variant& variant) {
write_return_value(get_and_rewind_buffer(p_env), variant);
ktvariant::send_variant_to_buffer(variant, get_and_rewind_buffer(p_env));
}

void TransferContext::icall(JNIEnv* rawEnv, jobject instance, jlong j_ptr, jint p_method_index, jint expectedReturnType) {
Expand Down Expand Up @@ -121,7 +121,7 @@ void TransferContext::icall(JNIEnv* rawEnv, jobject instance, jlong j_ptr, jint
const Variant& ret_value {methodBind->call(ptr, args_ptr, args_size, r_error)};

buffer->rewind();
write_return_value(buffer, ret_value);
ktvariant::send_variant_to_buffer(ret_value, buffer);
} else {
Variant* args {variant_args + stack_offset};
read_args_to_array(buffer, args, args_size);
Expand All @@ -134,7 +134,7 @@ void TransferContext::icall(JNIEnv* rawEnv, jobject instance, jlong j_ptr, jint
stack_offset -= args_size;

buffer->rewind();
write_return_value(buffer, ret_value);
ktvariant::send_variant_to_buffer(ret_value, buffer);
}

#ifdef DEBUG_ENABLED
Expand Down
4 changes: 0 additions & 4 deletions src/memory/transfer_context.h
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,6 @@ class TransferContext : public JavaInstanceWrapper {
}
}

_FORCE_INLINE_ static void write_return_value(SharedBuffer* buffer, const Variant& r_ret) {
ktvariant::send_variant_to_buffer(r_ret, buffer);
}

// clang-format off
DECLARE_JNI_METHODS(
JNI_METHOD(GET_BUFFER, "getBuffer", "()Ljava/nio/ByteBuffer;")
Expand Down

0 comments on commit 474a8a6

Please sign in to comment.