[filters] Stripping parenthesis whitespace

Author: John Bodley

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)
 

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)