Merge pull request #506 from Crozzers/fix-505

Fix `_uniform_outdent` failing with empty strings (issue #505)

CHANGES.md

@@ -4,6 +4,7 @@
 
 - [pull #500] Add `<thead>` tag to html-classes extra
 - [pull #502] Replace deprecated `optparse` with `argparse`
+- [pull #506] Fix `_uniform_outdent` failing with empty strings (issue #505)
 
 
 ## python-markdown2 2.4.8

lib/markdown2.py

@@ -2609,12 +2609,17 @@ def _uniform_outdent(self, text, min_outdent=None, max_outdent=None):
             re.findall(r'^[ \t]*', line)[0] if line else None
             for line in text.splitlines()
         ]
+        whitespace_not_empty = [i for i in whitespace if i is not None]
+
+        # if no whitespace detected (ie: no lines in code block, issue #505)
+        if not whitespace_not_empty:
+            return '', text
 
         # get minimum common whitespace
-        outdent = min(i for i in whitespace if i is not None)
+        outdent = min(whitespace_not_empty)
         # adjust min common ws to be within bounds
         if min_outdent is not None:
-            outdent = min([i for i in whitespace if i is not None and i >= min_outdent] or [min_outdent])
+            outdent = min([i for i in whitespace_not_empty if i >= min_outdent] or [min_outdent])
         if max_outdent is not None:
             outdent = min(outdent, max_outdent)
 

test/tm-cases/empty_fenced_code_blocks.html

@@ -0,0 +1,20 @@
+<div class="codehilite">
+<pre><span></span><code>
+</code></pre>
+</div>
+
+<pre><code>
+</code></pre>
+
+<p>Pygments removes the empty line whitespace from the next code block</p>
+
+<div class="codehilite">
+<pre><span></span><code>
+</code></pre>
+</div>
+
+<pre><code>
+
+
+
+</code></pre>

test/tm-cases/empty_fenced_code_blocks.opts

@@ -0,0 +1 @@
+{"extras": ["fenced-code-blocks"]}

test/tm-cases/empty_fenced_code_blocks.text

@@ -0,0 +1,20 @@
+```shell
+```
+
+```
+```
+
+Pygments removes the empty line whitespace from the next code block
+```shell
+
+
+
+
+```
+
+```
+
+
+
+
+```