Merge pull request #3010 from jfbu/fixcodeinparsedliteral

Fix #3009: Bad rendering of parsed-literals in LaTeX

CHANGES

@@ -21,7 +21,7 @@ Bugs fixed
 * #2990: linkcheck raises "Can't convert 'bytes' object to str implicitly" error
   if linkcheck_anchors enabled
 * #3004: Invalid link types "top" and "up" are used
-
+* #3009: Bad rendering of parsed-literals in LaTeX
 
 Documentation
 -------------

sphinx/texinputs/sphinx.sty

@@ -160,10 +160,13 @@
 % Some custom font markup commands.
 % *** the macros without \sphinx prefix are still defined at bottom of file ***
 \newcommand{\sphinxstrong}[1]{{\textbf{#1}}}
-% let \sphinxcode and \sphinxbfcode use straight quotes. \@noligs patched by upquote,
-% but needs protection in "moving arguments" such as for captions.
-% Use \scantokens to handle e.g. \item[{\sphinxcode{'fontenc'}}]
-\DeclareRobustCommand{\sphinxcode}[1]{{\@noligs\scantokens{\texttt{#1}\relax}}}
+% to obtain straight quotes we execute \@noligs as patched by upquote, the
+% macro must be robust in case it is used in captions e.g., and \scantokens is
+% needed in such cases and others such as \item[{\sphinxcode{'fontenc'}}]
+% in 'alltt' \@noligs is done already, and the \scantokens must be avoided.
+\DeclareRobustCommand{\sphinxcode}[1]{{\def\@tempa{alltt}%
+  \ifx\@tempa\@currenvir\else\@noligs\endlinechar\m@ne\everyeof{\noexpand}%
+  \expandafter\scantokens\fi {\texttt{#1}}}}
 \newcommand{\sphinxbfcode}[1]{\sphinxcode{\bfseries#1}}
 \newcommand{\sphinxemail}[1]{\textsf{#1}}
 \newcommand{\sphinxtablecontinued}[1]{\textsf{#1}}
@@ -994,8 +997,12 @@
 % by default, also define macros with the no-prefix names
 \ifsphinxKeepOldNames
   \typeout{** (sphinx) defining (legacy) text style macros without \string\sphinx\space prefix}
-  \typeout{** if clashes with packages, set latex_keep_old_macro_names=False in conf.py}
-  \@for\@tempa:=strong,bfcode,email,tablecontinued,titleref,%
+  \typeout{** if clashes with packages, set latex_keep_old_macro_names=False
+    in conf.py}
+  % robustified case needs special treatment
+  \newcommand\code{}\DeclareRobustCommand{\code}{}%
+  \expandafter\let\csname code \endcsname\relax\def\sphinxcode{\code}%
+  \@for\@tempa:=code ,strong,bfcode,email,tablecontinued,titleref,%
                 menuselection,accelerator,crossref,termref,optional\do
 {% first, check if command with no prefix already exists
   \expandafter\newcommand\csname\@tempa\endcsname{}%
@@ -1005,11 +1012,7 @@
   % redefine the \sphinx prefixed macro to expand to non-prefixed one
   \expandafter\def\csname sphinx\@tempa\expandafter\endcsname
                   \expandafter{\csname\@tempa\endcsname}%
-}
-  % robustified case needs special treatment
-  \newcommand\code{}\let\code\relax
-  \DeclareRobustCommand{\code}[1]{{\@noligs\scantokens{\texttt{#1}\relax}}}
-  \def\sphinxcode{\code}%
+}%
 \fi
 
 % additional customizable styling