Skip to content

Commit

Permalink
Fix patching of commands that contain parameter tokens
Browse files Browse the repository at this point in the history
Fixes #697
  • Loading branch information
PhelypeOleinik committed Oct 21, 2021
1 parent 64ae62e commit 4932d3d
Show file tree
Hide file tree
Showing 4 changed files with 334 additions and 75 deletions.
5 changes: 5 additions & 0 deletions base/changes.txt
Expand Up @@ -6,6 +6,11 @@ completeness or accuracy and it contains some references to files that
are not part of the distribution.
================================================================================

2021-10-20 Phelype Oleinik <phelype.oleinik@latex-project.org>

* ltcmdhooks.dtx (subsubsection{Patching by expansion and redefinition}):
Fix patching of commands that contain parameter tokens (gh/697)

2021-10-19 Frank Mittelbach <Frank.Mittelbach@latex-project.org>

* ltpara.dtx (subsection{The error messages}):
Expand Down
17 changes: 17 additions & 0 deletions base/doc/ltnews34.tex
Expand Up @@ -320,6 +320,19 @@ \subsection{Changed how \cs{RemoveFromHook} treats code that isn't in the hook}



\subsection{Fix patching of commands that contain parameter tokens}

In the previous release, \LaTeX's hook mechanism was extended to add
support for hooking into commands using generic \hook{cmd} hooks
(see~\cite{34:ltnews33}). The initial version had a bug, in which the
patching of some commands with a parameter token (normally |#|) in their
definition would fail with a low-level \TeX{} error. The bug has been
fixed, and now patching works for those commands as well.
%
\githubissue{697}




\subsection{???}

Expand Down Expand Up @@ -796,6 +809,10 @@ \subsection{???}
\emph{\LaTeXe{} news 32}.\\
\url{https://latex-project.org/news/latex2e-news/ltnews32.pdf}

\bibitem{34:ltnews33} \LaTeX{} Project Team:
\emph{\LaTeXe{} news 33}.\\
\url{https://latex-project.org/news/latex2e-news/ltnews33.pdf}

\end{thebibliography}


Expand Down

0 comments on commit 4932d3d

Please sign in to comment.