Fix spurious space after unary expression

This happened when the operand was a complex expression.

Fixes #15

diff --git a/README.md b/README.md
index 63fb4b2e58eeac2a3eac8d53c2a97dc0d78ac53d..c0870a8d748ccf49571f2c7ccfab3c4cda62b73a 100644 (file)
--- a/README.md
+++ b/README.md
@@ -257,6 +257,12 @@ You can still try but prepare to be disappointed.
 
 * fixed spurious space in parenthesized set expressions (#7)
 
 
 * fixed spurious space in parenthesized set expressions (#7)
 

+* fixed spurious space after opening parentheses and in default
+  arguments (#14, #17)
+
+* fixed spurious space after unary operators when the operand was
+  a complex expression (#15)
+

 
 ### 18.3a0
 
 
 ### 18.3a0
 

diff --git a/black.py b/black.py
index 75fbdcadd41159f0b38446288f1b05c5f1408995..d48fddbe6526d6dc2753a71c482e53afe8c2f9ee 100644 (file)
--- a/black.py
+++ b/black.py
@@ -815,6 +815,9 @@ def whitespace(leaf: Leaf) -> str:
             if prevp.parent and prevp.parent.type == syms.subscript:
                 return NO
 
             if prevp.parent and prevp.parent.type == syms.subscript:
                 return NO
 

+        elif prevp.parent and prevp.parent.type == syms.factor:
+            return NO
+

     elif prev.type in OPENING_BRACKETS:
         return NO
 
     elif prev.type in OPENING_BRACKETS:
         return NO
 

diff --git a/tests/expression.py b/tests/expression.py
index 1d9f0ab57ade05f8f35ef02b04962e6c6673da31..68707498aefe2d0f3310526c0fbac668da617091 100644 (file)
--- a/tests/expression.py
+++ b/tests/expression.py
@@ -31,6 +31,7 @@ not great
 -1
 ~int and not v1 ^ 123 + v2 | True
 (~int) and (not ((v1 ^ (123 + v2)) | True))
 -1
 ~int and not v1 ^ 123 + v2 | True
 (~int) and (not ((v1 ^ (123 + v2)) | True))

+flags & ~ select.EPOLLIN and waiters.write_task is not None

 lambda arg: None
 lambda a=True: a
 lambda a, b, c=True: a
 lambda arg: None
 lambda a=True: a
 lambda a, b, c=True: a


@@ -155,6 +156,7 @@ not great
 -1
 ~int and not v1 ^ 123 + v2 | True
 (~int) and (not ((v1 ^ (123 + v2)) | True))
 -1
 ~int and not v1 ^ 123 + v2 | True
 (~int) and (not ((v1 ^ (123 + v2)) | True))

+flags & ~select.EPOLLIN and waiters.write_task is not None

 lambda arg: None
 lambda a=True: a
 lambda a, b, c=True: a
 lambda arg: None
 lambda a=True: a
 lambda a, b, c=True: a