Fix viewbox whitespace processing.

scour/scour.py

@@ -3390,7 +3390,9 @@ def properlySizeDoc(docElement, options):
     # else we have a statically sized image and we should try to remedy that
 
     # parse viewBox attribute
-    vbSep = RE_COMMA_WSP.split(docElement.getAttribute('viewBox'))
+    viewBox = docElement.getAttribute('viewBox')
+    lengths = RE_COMMA_WSP.split(viewBox)
+    vbSep = [length for length in lengths if length.strip()]
     # if we have a valid viewBox we need to check it
     if len(vbSep) == 4:
         try:
@@ -3819,7 +3821,7 @@ def xmlnsUnused(prefix, namespace):
     viewBox = doc.documentElement.getAttribute('viewBox')
     if viewBox:
         lengths = RE_COMMA_WSP.split(viewBox)
-        lengths = [scourUnitlessLength(length) for length in lengths]
+        lengths = [scourUnitlessLength(length) for length in lengths if length.strip()]
         doc.documentElement.setAttribute('viewBox', ' '.join(lengths))
 
     # more length scouring in this function

test_scour.py

@@ -2786,6 +2786,13 @@ def test_viewbox_remove_width_and_height(self):
         self.assertEqual(width, '', "width not removed with '--enable-viewboxing'.")
         self.assertEqual(height, '', "height not removed with '--enable-viewboxing'.")
 
+    def test_viewbox_remove_width_and_height_ws(self):
+        doc = scourXmlFile('unittests/viewbox-remove-ws.svg', parse_args(['--enable-viewboxing']))
+        width = doc.documentElement.getAttribute('width')
+        height = doc.documentElement.getAttribute('height')
+        self.assertEqual(width, '', "width not removed with '--enable-viewboxing'.")
+        self.assertEqual(height, '', "height not removed with '--enable-viewboxing'.")
+
 
 # TODO: write tests for --keep-editor-data