From e2610138a75da9f5216c198a5bab67a65c2746c7 Mon Sep 17 00:00:00 2001 From: Github Executorch Date: Thu, 28 Aug 2025 12:19:29 -0700 Subject: [PATCH] Summary: Throw Executorch exception which is mapped to underlying native error code Test Plan: Tested with executorch-examples Reviewers: Subscribers: Tasks: Tags: --- extension/android/jni/jni_layer.cpp | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/extension/android/jni/jni_layer.cpp b/extension/android/jni/jni_layer.cpp index 7ad54ffc360..e7ef6e62c74 100644 --- a/extension/android/jni/jni_layer.cpp +++ b/extension/android/jni/jni_layer.cpp @@ -308,9 +308,7 @@ class ExecuTorchJni : public facebook::jni::HybridClass { << std::uppercase << static_cast(result) << "]"; jni_helper::throwExecutorchException( - static_cast( - Error::InvalidArgument), // For backward compatibility - ss.str()); + static_cast(result), ss.str()); return {}; } auto&& underlying_method = module_->methods_[method].method; @@ -460,9 +458,7 @@ class ExecuTorchJni : public facebook::jni::HybridClass { << "]"; jni_helper::throwExecutorchException( - static_cast( - Error::InvalidArgument), // For backward compatibility - ss.str()); + static_cast(Error::InvalidArgument), ss.str()); return {}; } const auto& methods = names_result.get();