final cleanup

Author: cristianoc

CHANGELOG.md

@@ -25,7 +25,7 @@
 - Remove `Stdlib_Char` module for now. https://github.com/rescript-lang/rescript/pull/7367
 - Convert internal JavaScript codebase into ESM, ReScript package itself is now ESM (`"type": "module"`). https://github.com/rescript-lang/rescript/pull/6899
 - Add built-in support for the JavaScript `in` operator. https://github.com/rescript-lang/rescript/pull/7342
-- AST cleanup: add `Pexp_await` ast node instead of `res.await` attribute. (The attribute is still used for await on modules currently).
+- AST cleanup: add `Pexp_await` ast node instead of `res.await` attribute. (The attribute is still used for await on modules currently). https://github.com/rescript-lang/rescript/pull/7368
 
 #### :nail_care: Polish
 

compiler/syntax/src/res_printer.ml

@@ -3756,14 +3756,8 @@ and print_binary_expression ~state (expr : Parsetree.expression) cmt_tbl =
               | [] -> doc
               | _ -> add_parens doc
             in
-            let is_await =
-              (match expr.pexp_desc with
-              | Pexp_await _ -> true
-              | _ -> false)
-              || ParsetreeViewer.expr_is_await expr
-            in
             let doc =
-              if is_await then
+              if ParsetreeViewer.expr_is_await expr then
                 let parens =
                   Res_parens.binary_operator_inside_await_needs_parens operator
                 in