quickjs-tart

commit 23b04b95a974867c0e5fbfd76a38eb8ac9e4807a
parent 792c9b43d374906770eb7d8ccede8240d03a7ac1
Author: Iván Ávalos <avalos@disroot.org>
Date:   Fri, 14 Jun 2024 10:17:50 -0600

android: fix req_create call invocation

Diffstat:
M
QuickJS-android/qtart/src/main/java/net/taler/qtart/Networking.kt
 | 
4
+++-
M
QuickJS-android/qtart/src/main/java/net/taler/qtart/TalerWalletCore.kt
 | 
9
++++-----

2 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/QuickJS-android/qtart/src/main/java/net/taler/qtart/Networking.kt b/QuickJS-android/qtart/src/main/java/net/taler/qtart/Networking.kt
@@ -71,7 +71,7 @@ object Networking {
     internal fun httpCreateRequest(
         reqInfo: JSHttpRequestInfo,
         reqHandler: RequestHandler,
-    ) {
+    ): Int {
         val requestId = lastRequestId.addAndGet(1)
 
         reqHandler.handleRequest(
@@ -109,6 +109,8 @@ object Networking {
             Log.d("Networking", "Response ready to send to Qtart: $rawResp")
             reqInfo.response_cb?.invoke(reqInfo.response_cb_cls!!, rawResp)
         }
+
+        return requestId
     }
 
     /**
diff --git a/QuickJS-android/qtart/src/main/java/net/taler/qtart/TalerWalletCore.kt b/QuickJS-android/qtart/src/main/java/net/taler/qtart/TalerWalletCore.kt
@@ -198,11 +198,10 @@ class TalerWalletCore {
     fun setHttpClient(handler: Networking.RequestHandler) {
         this.currentReqCreateHandler = object: TalerNative.JSHttpReqCreateFn {
             override fun invoke(cls: Pointer, req_info: TalerNative.JSHttpRequestInfo): Int {
-                Networking.httpCreateRequest(req_info, handler)
-                // TODO: return positive number on success, negative on failure
-                //  right now, this can't be done synchronously; doing it
-                //  async might require a modification in the C API.
-                return 1
+                // TODO: return positive request_id on success, negative on failure
+                //  right now, this can't be done synchronously; doing it async
+                //  might require a modification in the C API.
+                return Networking.httpCreateRequest(req_info, handler)
             }
         }
         this.currentReqCancelHandler = object: TalerNative.JSHttpReqCancelFn {