[filters] Stripping parenthesis whitespace

John Bodley · andialbrecht · commit 5faadeba8e33 · 2019-03-10T08:09:46.000+01:00
diff --git a/sqlparse/filters/others.py b/sqlparse/filters/others.py
@@ -76,9 +76,9 @@ def _stripws_identifierlist(self, tlist):
         return self._stripws_default(tlist)
 
     def _stripws_parenthesis(self, tlist):
-        if tlist.tokens[1].is_whitespace:
+        while tlist.tokens[1].is_whitespace:
             tlist.tokens.pop(1)
-        if tlist.tokens[-2].is_whitespace:
+        while tlist.tokens[-2].is_whitespace:
             tlist.tokens.pop(-2)
         self._stripws_default(tlist)
 
diff --git a/tests/test_format.py b/tests/test_format.py
@@ -377,9 +377,9 @@ def test_parenthesis(self):
             '   from foo);'])
         assert f("select f(1)") == 'select f(1)'
         assert f("select f( 1 )") == 'select f(1)'
-        assert f("select f(\n\n\n1\n\n\n)") == 'select f( 1 )'
-        assert f("select f(\n\n\n 1 \n\n\n)") == 'select f( 1 )'
-        assert f("select f(\n\n\n  1  \n\n\n)") == 'select f( 1 )'
+        assert f("select f(\n\n\n1\n\n\n)") == 'select f(1)'
+        assert f("select f(\n\n\n 1 \n\n\n)") == 'select f(1)'
+        assert f("select f(\n\n\n  1  \n\n\n)") == 'select f(1)'
 
     def test_where(self):
         f = lambda sql: sqlparse.format(sql, reindent=True)
