From 2c9eeaf9f46466b13583d1a09691a48ded639ba9 Mon Sep 17 00:00:00 2001
From: Kai Ninomiya <kainino@chromium.org>
Date: Wed, 6 Mar 2024 20:33:34 -0800
Subject: [PATCH] Avoid generating empty statement in receiveI64ParamAsI53

Fixes a few Closure lint warnings. (Though of course there are very many
other Closure warnings.)
---
 src/parseTools.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/parseTools.js b/src/parseTools.js
index fa89005f0c65b..636ced683775e 100644
--- a/src/parseTools.js
+++ b/src/parseTools.js
@@ -911,14 +911,14 @@ function defineI64Param(name) {
 
 
 function receiveI64ParamAsI53(name, onError, handleErrors = true) {
-  var errorHandler = handleErrors ? `if (isNaN(${name})) return ${onError}` : '';
+  var errorHandler = handleErrors ? `if (isNaN(${name})) { return ${onError}; }` : '';
   if (WASM_BIGINT) {
     // Just convert the bigint into a double.
-    return `${name} = bigintToI53Checked(${name});${errorHandler};`;
+    return `${name} = bigintToI53Checked(${name});${errorHandler}`;
   }
   // Convert the high/low pair to a Number, checking for
   // overflow of the I53 range and returning onError in that case.
-  return `var ${name} = convertI32PairToI53Checked(${name}_low, ${name}_high);${errorHandler};`;
+  return `var ${name} = convertI32PairToI53Checked(${name}_low, ${name}_high);${errorHandler}`;
 }
 
 function receiveI64ParamAsI53Unchecked(name) {