unnecessary parens in while with assign

Prettier 1.14.2 Playground link

--parser babylon

Input:

while ($var = 1) {}

Output:

while (($var = 1)) {}

Expected behavior:

while ($var = 1) {}

---

Edge case.

[vjeux]

This is to prevent the mistake of trying to do “a == 1” but writing “a = 1” and subtly getting it wrong. If you see a double parenthesis then it’s obvious that you are doing an assignment.

[lydell]

As far as I remember, this is intended to show that you actually intended to do assignment and not comparison. See also: https://eslint.org/docs/rules/no-cond-assign