Js output: fix extra newline after if

CHANGELOG.md

@@ -29,6 +29,7 @@
 - Keep track of compiler info during build. https://github.com/rescript-lang/rescript/pull/7889
 - Improve option optimization for constants. https://github.com/rescript-lang/rescript/pull/7913
 - Option optimization: do not create redundant local vars. https://github.com/rescript-lang/rescript/pull/7915
+- Js output: remove superfluous newline after every `if`. https://github.com/rescript-lang/rescript/pull/7920
 
 #### :house: Internal
 

compiler/core/js_dump.ml

@@ -1365,9 +1365,7 @@ and statement_desc top cxt f (s : J.statement_desc) : cxt =
     P.space f;
     let cxt = brace_block cxt f s1 in
     match s2 with
-    | [] | [{statement_desc = Block [] | Exp {expression_desc = Var _}}] ->
-      P.newline f;
-      cxt
+    | [] | [{statement_desc = Block [] | Exp {expression_desc = Var _}}] -> cxt
     | [({statement_desc = If _} as nest)]
     | [{statement_desc = Block [({statement_desc = If _; _} as nest)]; _}] ->
       P.space f;

packages/@rescript/runtime/lib/es6/Belt_Array.js

@@ -7,7 +7,6 @@ function get(arr, i) {
   if (i >= 0 && i < arr.length) {
     return Primitive_option.some(arr[i]);
   }
-
 }
 
 function getOrThrow(arr, i) {
@@ -328,7 +327,6 @@ function keep(a, f) {
       r[j] = v;
       j = j + 1 | 0;
     }
-
   }
   r.length = j;
   return r;
@@ -344,7 +342,6 @@ function keepWithIndex(a, f) {
       r[j] = v;
       j = j + 1 | 0;
     }
-
   }
   r.length = j;
   return r;
@@ -361,7 +358,6 @@ function keepMap(a, f) {
       r[j] = Primitive_option.valFromOption(v$1);
       j = j + 1 | 0;
     }
-
   }
   r.length = j;
   return r;

packages/@rescript/runtime/lib/es6/Belt_HashMap.js

@@ -57,7 +57,6 @@ function set0(h, key, value, eq, hash) {
       };
       h.size = h.size + 1 | 0;
     }
-
   } else {
     h_buckets[i] = {
       key: key,
@@ -84,11 +83,9 @@ function set0(h, key, value, eq, hash) {
       if (tail !== undefined) {
         tail.next = undefined;
       }
-
     }
     return;
   }
-
 }
 
 function set(h, key, value) {
@@ -166,7 +163,6 @@ function get(h, key) {
       };
     }
   }
-
 }
 
 function has(h, key) {

packages/@rescript/runtime/lib/es6/Belt_HashMapInt.js

@@ -54,7 +54,6 @@ function set(h, key, value) {
       };
       h.size = h.size + 1 | 0;
     }
-
   } else {
     h_buckets[i] = {
       key: key,
@@ -81,11 +80,9 @@ function set(h, key, value) {
       if (tail !== undefined) {
         tail.next = undefined;
       }
-
     }
     return;
   }
-
 }
 
 function remove(h, key) {
@@ -118,7 +115,6 @@ function remove(h, key) {
       };
     }
   }
-
 }
 
 function get(h, key) {
@@ -157,7 +153,6 @@ function get(h, key) {
       };
     }
   }
-
 }
 
 function has(h, key) {

packages/@rescript/runtime/lib/es6/Belt_HashMapString.js

@@ -54,7 +54,6 @@ function set(h, key, value) {
       };
       h.size = h.size + 1 | 0;
     }
-
   } else {
     h_buckets[i] = {
       key: key,
@@ -81,11 +80,9 @@ function set(h, key, value) {
       if (tail !== undefined) {
         tail.next = undefined;
       }
-
     }
     return;
   }
-
 }
 
 function remove(h, key) {
@@ -118,7 +115,6 @@ function remove(h, key) {
       };
     }
   }
-
 }
 
 function get(h, key) {
@@ -157,7 +153,6 @@ function get(h, key) {
       };
     }
   }
-
 }
 
 function has(h, key) {

packages/@rescript/runtime/lib/es6/Belt_HashSet.js

@@ -111,11 +111,9 @@ function add0(h, key, hash, eq) {
       if (tail !== undefined) {
         tail.next = undefined;
       }
-
     }
     return;
   }
-
 }
 
 function add(h, key) {

packages/@rescript/runtime/lib/es6/Belt_HashSetInt.js

@@ -111,11 +111,9 @@ function add(h, key) {
       if (tail !== undefined) {
         tail.next = undefined;
       }
-
     }
     return;
   }
-
 }
 
 function has(h, key) {

packages/@rescript/runtime/lib/es6/Belt_HashSetString.js

@@ -111,11 +111,9 @@ function add(h, key) {
       if (tail !== undefined) {
         tail.next = undefined;
       }
-
     }
     return;
   }
-
 }
 
 function has(h, key) {

packages/@rescript/runtime/lib/es6/Belt_List.js

@@ -8,7 +8,6 @@ function head(x) {
   if (x !== 0) {
     return Primitive_option.some(x.hd);
   }
-
 }
 
 function headOrThrow(x) {
@@ -25,7 +24,6 @@ function tail(x) {
   if (x !== 0) {
     return x.tl;
   }
-
 }
 
 function tailOrThrow(x) {
@@ -445,7 +443,6 @@ function take(lst, n) {
   if (has) {
     return cell;
   }
-
 }
 
 function drop(lst, n) {
@@ -494,7 +491,6 @@ function splitAt(lst, n) {
       rest
     ];
   }
-
 }
 
 function concat(xs, ys) {

packages/@rescript/runtime/lib/es6/Belt_MapDict.js

@@ -121,7 +121,6 @@ function remove(n, x, cmp) {
   if (n !== undefined) {
     return removeAux0(n, x, cmp);
   }
-
 }
 
 function mergeMany(h, arr, cmp) {
@@ -255,7 +254,6 @@ function removeMany(t, keys, cmp) {
       continue;
     };
   }
-
 }
 
 let empty;

packages/@rescript/runtime/lib/es6/Belt_MapInt.js

@@ -113,7 +113,6 @@ function remove(n, x) {
   if (n !== undefined) {
     return removeAux(n, x);
   }
-
 }
 
 function removeMany(t, keys) {
@@ -137,7 +136,6 @@ function removeMany(t, keys) {
       continue;
     };
   }
-
 }
 
 function mergeMany(h, arr) {

packages/@rescript/runtime/lib/es6/Belt_MapString.js

@@ -113,7 +113,6 @@ function remove(n, x) {
   if (n !== undefined) {
     return removeAux(n, x);
   }
-
 }
 
 function removeMany(t, keys) {
@@ -137,7 +136,6 @@ function removeMany(t, keys) {
       continue;
     };
   }
-
 }
 
 function mergeMany(h, arr) {

packages/@rescript/runtime/lib/es6/Belt_MutableMap.js

@@ -50,7 +50,6 @@ function remove(d, k) {
     d.data = newRoot;
     return;
   }
-
 }
 
 function removeArrayMutateAux(_t, xs, _i, len, cmp) {
@@ -82,7 +81,6 @@ function removeMany(d, xs) {
     d.data = newRoot;
     return;
   }
-
 }
 
 function updateDone(t, x, f, cmp) {
@@ -132,7 +130,6 @@ function update(t, x, f) {
     t.data = newRoot;
     return;
   }
-
 }
 
 function make(id) {
@@ -280,7 +277,6 @@ function set(m, e, v) {
     m.data = newRoot;
     return;
   }
-
 }
 
 function mergeManyAux(t, xs, cmp) {
@@ -299,7 +295,6 @@ function mergeMany(d, xs) {
     d.data = newRoot;
     return;
   }
-
 }
 
 let Int;

packages/@rescript/runtime/lib/es6/Belt_MutableMapInt.js

@@ -58,7 +58,6 @@ function set(m, k, v) {
     m.data = v$1;
     return;
   }
-
 }
 
 function forEach(d, f) {
@@ -161,7 +160,6 @@ function remove(d, v) {
     d.data = newRoot;
     return;
   }
-
 }
 
 function updateDone(t, x, f) {
@@ -211,7 +209,6 @@ function update(t, x, f) {
     t.data = newRoot;
     return;
   }
-
 }
 
 function removeArrayMutateAux(_t, xs, _i, len) {
@@ -243,7 +240,6 @@ function removeMany(d, xs) {
     d.data = newRoot;
     return;
   }
-
 }
 
 function fromArray(xs) {

packages/@rescript/runtime/lib/es6/Belt_MutableMapString.js

@@ -58,7 +58,6 @@ function set(m, k, v) {
     m.data = v$1;
     return;
   }
-
 }
 
 function forEach(d, f) {
@@ -161,7 +160,6 @@ function remove(d, v) {
     d.data = newRoot;
     return;
   }
-
 }
 
 function updateDone(t, x, f) {
@@ -211,7 +209,6 @@ function update(t, x, f) {
     t.data = newRoot;
     return;
   }
-
 }
 
 function removeArrayMutateAux(_t, xs, _i, len) {
@@ -243,7 +240,6 @@ function removeMany(d, xs) {
     d.data = newRoot;
     return;
   }
-
 }
 
 function fromArray(xs) {

packages/@rescript/runtime/lib/es6/Belt_MutableQueue.js

@@ -38,15 +38,13 @@ function peek(q) {
   if (v !== undefined) {
     return Primitive_option.some(v.content);
   }
-
 }
 
 function peekUndefined(q) {
   let v = q.first;
   if (v !== undefined) {
     return v.content;
   }
-
 }
 
 function peekOrThrow(q) {