From b1770039f4d483cdab2d4d136cd7ffbf12f165b7 Mon Sep 17 00:00:00 2001
From: Sebastiano Tronto <sebastiano@tronto.net>
Date: Wed, 28 May 2025 14:32:38 +0200
Subject: Enable WASM64

---
 web/callback.js       |  9 ++++++++-
 web/http/index.html   |  2 +-
 web/http/nissyapp.mjs | 14 --------------
 web/storage.cpp       |  7 +++++--
 web/worker.mjs        |  7 ++-----
 5 files changed, 16 insertions(+), 23 deletions(-)

(limited to 'web')

diff --git a/web/callback.js b/web/callback.js
index 178b8e9..e277cf5 100644
--- a/web/callback.js
+++ b/web/callback.js
@@ -31,8 +31,15 @@ addCallbackFunction: function(f) {
 
 callFunction__deps: [ 'cbfl', 'validateCallbackId' ],
 callFunction: function(id, arg) {
+  // This is a workaround related to usign WASM64
+  // JavaScript's UTF8ToString expects a pointer argument, which for JS is
+  // of type "number", but WASM64 is passing a BigInt. See also:
+  // https://github.com/emscripten-core/emscripten/issues/21541
+  // (but I could not make the suggested solution work in this case).
+  // TODO: check if there is a better workaround.
+  const non64_arg = Number(arg);
   if (_validateCallbackId(id))
-    _cbfl[id](UTF8ToString(arg));
+    _cbfl[id](UTF8ToString(non64_arg));
 },
 
 callFunctionInt__deps: [ 'cbfl', 'validateCallbackId' ],
diff --git a/web/http/index.html b/web/http/index.html
index 0846ea8..47ccc0d 100644
--- a/web/http/index.html
+++ b/web/http/index.html
@@ -11,7 +11,7 @@
     <select id="solverSelector">
       <option value="h48h0k4">h48 h=0 k=4 (59Mb)</option>
       <option value="h48h3k2" selected="selected">h48 h=3 k=2 (283Mb)</option>
-      <!--option value="h48h7k2">h48 h=7 k=2 (3.6Gb)</option-->
+      <option value="h48h7k2">h48 h=7 k=2 (3.6Gb)</option>
     </select>
     <button id="solveButton">Solve</button>
     <div id="confirmDownload" style="display:none">
diff --git a/web/http/nissyapp.mjs b/web/http/nissyapp.mjs
index b8a525c..55ca198 100644
--- a/web/http/nissyapp.mjs
+++ b/web/http/nissyapp.mjs
@@ -50,20 +50,6 @@ solveButton.addEventListener("click", () => {
   const solver = solverSelector.options[solverSelector.selectedIndex].value;
   const scramble = scrField.value;
   loadDataThenSolve(solver, scramble);
-
-  const solveCallbackId = ++lastCallbackId;
-  callbacks.set(solveCallbackId, {
-    f: (callbackArg, loadResult) => {
-      if (loadResult.success) {
-        startSolve(callbackArg.solver, callbackArg.scramble);
-      } else {
-        updateResults("", "", true);
-      }
-    },
-    arg: { solver: solver, scramble: scramble }
-  });
-
-  const downloadCallbackId = ++lastCallbackId;
 });
 
 function loadDataThenSolve(solver, scramble) {
diff --git a/web/storage.cpp b/web/storage.cpp
index cbf9e9c..eef145e 100644
--- a/web/storage.cpp
+++ b/web/storage.cpp
@@ -43,8 +43,11 @@ EM_ASYNC_JS(int, download_and_store, (const char *key, const char *url), {
 	    self instanceof WorkerGlobalScope;
 	console.assert(inBrowser || inWorker, "Non-browsers not supported");
 
-	url = UTF8ToString(url);
-	key = UTF8ToString(key);
+	// See comment in callback.js about this workaround
+	const non64_url = Number(url);
+	const non64_key = Number(key);
+	url = UTF8ToString(non64_url);
+	key = UTF8ToString(non64_key);
 	let response = await fetch(url);
 	if (!response.ok) {
 		console.log("Error downloading data for " + key);
diff --git a/web/worker.mjs b/web/worker.mjs
index ceceaec..0cb618b 100644
--- a/web/worker.mjs
+++ b/web/worker.mjs
@@ -1,10 +1,7 @@
 import Nissy from "./nissy_web_module.mjs"
 
 const nissy = await Nissy();
-function log(message) {
-  postMessage({ command: "log", id: -1, object: message });
-  console.log(message);
-}
+const log = (msg) => postMessage({ command: "log", id: -1, object: msg });
 nissy.setLogger(nissy._addCallbackFunction(log));
 
 const commands = [
@@ -32,7 +29,7 @@ onmessage = (event) => {
     }
   }
  
-  log("[nissy worker] unknown command " + event.data.command);
+  console.log("[nissy worker] unknown command " + event.data.command);
 };
 
 // Load solver data from storage.
-- 
cgit v1.3