[latex3-commits] [git/LaTeX3-latex3-latex2e] gh479: Catcode setups must be outside \NewModuleRelease (7ec40542)

[PhelypeOleinik]

Repository : https://github.com/latex3/latex2e
On branch  : gh479
Link       : https://github.com/latex3/latex2e/commit/7ec40542d91f03fa7f0e291b6cf3a4b8d610c796

>---------------------------------------------------------------

commit 7ec40542d91f03fa7f0e291b6cf3a4b8d610c796
Author: PhelypeOleinik <phelype.oleinik at latex-project.org>
Date:   Fri Mar 19 23:42:58 2021 -0300

    Catcode setups must be outside \NewModuleRelease
    
    As I just found out by writing the documentation in latexrelease.dtx


>---------------------------------------------------------------

7ec40542d91f03fa7f0e291b6cf3a4b8d610c796
 base/ltcmd.dtx   |  9 +++------
 base/lthooks.dtx | 15 +++------------
 2 files changed, 6 insertions(+), 18 deletions(-)

diff --git a/base/ltcmd.dtx b/base/ltcmd.dtx
index 970181d1..68d0bd54 100644
--- a/base/ltcmd.dtx
+++ b/base/ltcmd.dtx
@@ -87,11 +87,8 @@
 %<*2ekernel|latexrelease>
 %<latexrelease>\edef\@latexrelease at catcode@null{\the\catcode`\^^@ }
 %<latexrelease>\catcode`\^^@=12
-%<latexrelease>\NewModuleRelease{ltcmd}{2021/05/01}%
-%    \end{macrocode}
-%
-%    \begin{macrocode}
 \ExplSyntaxOn
+%<latexrelease>\NewModuleRelease{ltcmd}{2021/05/01}%
 %    \end{macrocode}
 %
 % \subsection{Variables and constants}
@@ -4009,11 +4006,9 @@
 %    \end{macrocode}
 %
 %    \begin{macrocode}
-\ExplSyntaxOff
 %    \end{macrocode}
 %
 %    \begin{macrocode}
-%</2ekernel|latexrelease>
 %<latexrelease>
 %<latexrelease>\IncludeInRelease{2020/10/01}%
 %<latexrelease>                 {\NewDocumentCommand}{Document commands}%
@@ -4030,7 +4025,9 @@
 %<latexrelease>                 {\NewDocumentCommand}{Document commands}%
 %<latexrelease>
 %<latexrelease>\EndModuleRelease
+\ExplSyntaxOff
 %<latexrelease>\catcode`\^^@=\@latexrelease at catcode@null\relax
+%</2ekernel|latexrelease>
 %    \end{macrocode}
 %
 % We need to stop DocStrip treating |@@| in a special way at this point.
diff --git a/base/lthooks.dtx b/base/lthooks.dtx
index 3514964d..ffb2959a 100644
--- a/base/lthooks.dtx
+++ b/base/lthooks.dtx
@@ -1716,14 +1716,10 @@
 %         {Use \cs{NewModuleRelease}.}
 %    \begin{macrocode}
 %<*2ekernel|latexrelease>
-%<latexrelease>\NewModuleRelease{lthooks}{2020/10/01}
-%    \end{macrocode}
-%    
-%    \begin{macrocode}
 \ExplSyntaxOn
+%<latexrelease>\NewModuleRelease{lthooks}{2020/10/01}
 %    \end{macrocode}
 %
-%
 %  \subsection{Debugging}
 %
 %  \begin{macro}{\g_@@_debug_bool}
@@ -4466,12 +4462,6 @@
 % \InternalDetectionOn
 %  \end{macro}
 %
-%    
-%    \begin{macrocode}
-\ExplSyntaxOff
-%    \end{macrocode}
-%
-%
 %    Rolling back here doesn't undefine the interface commands as they
 %    may be used in packages without rollback functionality. So we
 %    just make them do nothing which may or may not work depending on
@@ -4479,7 +4469,6 @@
 % \changes{v1.0d}{2020/10/04}{Definition \cs{AddToHookNext} was supposed
 %                             to be for \cs{AddToHook} vice versa (gh/401)}
 %    \begin{macrocode}
-%</2ekernel|latexrelease>
 %
 %<latexrelease>\IncludeInRelease{0000/00/00}%
 %<latexrelease>                 {lthooks}{The hook management}%
@@ -4515,6 +4504,8 @@
 %<latexrelease>\long\def \IfHookEmptyTF #1#2#3{#2}
 %<latexrelease>
 %<latexrelease>\EndModuleRelease
+\ExplSyntaxOff
+%</2ekernel|latexrelease>
 %    \end{macrocode}
 %
 %