Make the `start-after and end-before options to the literalinclude` directive work correctly if not used together.

CHANGES

@@ -1,6 +1,9 @@
 Release 0.6.5 (in development)
 ==============================
 
+* Make the ``start-after`` and ``end-before`` options to the
+  ``literalinclude`` directive work correctly if not used together.
+
 * #321: Fix link generation in the LaTeX builder.
 
 

sphinx/directives/code.py

@@ -155,9 +155,9 @@ class LiteralInclude(Directive):
             use = not startafter
             res = []
             for line in lines:
-                if not use and startafter in line:
+                if not use and startafter and startafter in line:
                     use = True
-                elif use and endbefore in line:
+                elif use and endbefore and endbefore in line:
                     use = False
                     break
                 elif use:

tests/root/includes.txt

@@ -44,6 +44,12 @@ Literalinclude options
    :start-after: coding: utf-8
    :end-before: class Foo
 
+.. literalinclude:: literal.inc
+   :start-after: utf-8
+
+.. literalinclude:: literal.inc
+   :end-before: class Foo
+
 Test if dedenting before parsing works.
 
 .. highlight:: python

tests/test_markup.py

@@ -16,7 +16,6 @@ from util import *
 from docutils import frontend, utils, nodes
 from docutils.parsers import rst
 
-from sphinx import addnodes
 from sphinx.util import texescape
 from sphinx.writers.html import HTMLWriter, SmartyPantsHTMLTranslator
 from sphinx.writers.latex import LaTeXWriter, LaTeXTranslator