Test emitting let instead of var.

Fixes #856
rescript-lang · Apr 28, 2024 · 698fbf5 · 698fbf5
1 parent 17804a4
commit 698fbf5
Show file tree

Hide file tree

Showing 832 changed files with 21,319 additions and 21,319 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -29,6 +29,7 @@
 - Remove redundant space for empty return in generated js code. https://github.com/rescript-lang/rescript-compiler/pull/6745
 - Remove redundant space for export in generated js code. https://github.com/rescript-lang/rescript-compiler/pull/6560
 - Remove empty export blocks in generated js code. https://github.com/rescript-lang/rescript-compiler/pull/6744
+- In generated code, use `let` instead of `var`.
 
 # 11.1.0
 

diff --git a/jscomp/build_tests/custom_namespace/src/demo.bs.js b/jscomp/build_tests/custom_namespace/src/demo.bs.js
@@ -1,7 +1,7 @@
 'use strict';
 
 
-var v = 42;
+let v = 42;
 
 exports.v = v;
 /* No side effect */
diff --git a/jscomp/build_tests/devonly/src/demo.bs.js b/jscomp/build_tests/devonly/src/demo.bs.js
diff --git a/jscomp/build_tests/devonly/src/depdemo.bs.js b/jscomp/build_tests/devonly/src/depdemo.bs.js
diff --git a/jscomp/build_tests/devonly/src2/hello.bs.js b/jscomp/build_tests/devonly/src2/hello.bs.js
diff --git a/jscomp/build_tests/devonly/src2/hellodep.bs.js b/jscomp/build_tests/devonly/src2/hellodep.bs.js
diff --git a/jscomp/build_tests/ns/src/demo.js b/jscomp/build_tests/ns/src/demo.js
diff --git a/jscomp/build_tests/ns/src/hello.js b/jscomp/build_tests/ns/src/hello.js
diff --git a/jscomp/build_tests/react_ppx/src/React.bs.js b/jscomp/build_tests/react_ppx/src/React.bs.js
diff --git a/jscomp/build_tests/react_ppx/src/gpr_3695_test.bs.js b/jscomp/build_tests/react_ppx/src/gpr_3695_test.bs.js
diff --git a/jscomp/build_tests/react_ppx/src/gpr_3987_test.bs.js b/jscomp/build_tests/react_ppx/src/gpr_3987_test.bs.js
diff --git a/jscomp/build_tests/react_ppx/src/recursive_component_test.bs.js b/jscomp/build_tests/react_ppx/src/recursive_component_test.bs.js
diff --git a/jscomp/core/js_dump.ml b/jscomp/core/js_dump.ml
@@ -185,7 +185,7 @@ let pp_paren_params (inner_cxt : cxt) (f : Ext_pp.t) (lexical : Ident.t list) :
       id) *)
 
 let pp_var_assign cxt f id =
-  P.string f L.var;
+  P.string f L.let_;
   P.space f;
   let acxt = Ext_pp_scope.ident cxt f id in
   P.space f;
@@ -202,7 +202,7 @@ let pp_var_assign_this cxt f id =
   cxt
 
 let pp_var_declare cxt f id =
-  P.string f L.var;
+  P.string f L.let_;
   P.space f;
   let acxt = Ext_pp_scope.ident cxt f id in
   semi f;

diff --git a/jscomp/core/js_dump_import_export.ml b/jscomp/core/js_dump_import_export.ml
@@ -111,7 +111,7 @@ let requires require_lit cxt f (modules : (Ident.t * string * bool) list) =
   in
   P.at_least_two_lines f;
   Ext_list.rev_iter reversed_list (fun (s, file, default) ->
-      P.string f L.var;
+      P.string f L.let_;
       P.space f;
       P.string f s;
       P.space f;

diff --git a/jscomp/core/js_dump_lit.ml b/jscomp/core/js_dump_lit.ml
@@ -29,7 +29,7 @@ let function_ = "function"
 
 let function_async ~async = if async then "async function" else "function"
 
-let var = "var" (* should be able to switch to [let] easily*)
+let let_ = "let"
 
 let return = "return"
 

diff --git a/jscomp/gentype_tests/typescript-react-example/src/Wrapper.res.js b/jscomp/gentype_tests/typescript-react-example/src/Wrapper.res.js
diff --git a/jscomp/test/406_primitive_test.js b/jscomp/test/406_primitive_test.js
diff --git a/jscomp/test/AsInUncurriedExternals.js b/jscomp/test/AsInUncurriedExternals.js
diff --git a/jscomp/test/Coercion.js b/jscomp/test/Coercion.js
diff --git a/jscomp/test/DerivingAccessorsCurried.js b/jscomp/test/DerivingAccessorsCurried.js
diff --git a/jscomp/test/DerivingAccessorsUncurried.js b/jscomp/test/DerivingAccessorsUncurried.js