Correctly specify private variables as such
authorJoachim Breitner <mail@joachim-breitner.de>
Thu, 20 Jun 2013 20:01:50 +0000 (22:01 +0200)
committerJoachim Breitner <mail@joachim-breitner.de>
Thu, 20 Jun 2013 20:01:50 +0000 (22:01 +0200)
conteq.dtx

index d75aea3..0a66b11 100644 (file)
@@ -346,16 +346,16 @@ and the derived files           conteq.ins,
 % \subsection{Declaring local variables}
 %
 %    \begin{macrocode}
-\tl_new:N \l_conteq_default_layout_tl
-\tl_new:N \l_conteq_layout_tl
-\tl_new:N \l_conteq_body_tl
-\tl_new:N \l_conteq_lines_seq
-\tl_new:N \l_conteq_cells_seq
-\tl_new:N \l_conteq_head_tl
-\tl_new:N \l_conteq_lastline_tl
-\tl_new:N \l_conteq_rhs_tl
-\tl_new:N \l_conteq_expl_tl
-\tl_new:N \l_conteq_result_tl
+\tl_new:N \l__conteq_default_layout_tl
+\tl_new:N \l__conteq_layout_tl
+\tl_new:N \l__conteq_body_tl
+\tl_new:N \l__conteq_lines_seq
+\tl_new:N \l__conteq_cells_seq
+\tl_new:N \l__conteq_head_tl
+\tl_new:N \l__conteq_lastline_tl
+\tl_new:N \l__conteq_rhs_tl
+\tl_new:N \l__conteq_expl_tl
+\tl_new:N \l__conteq_result_tl
 %    \end{macrocode}
 
 % \subsection{Layouts}
@@ -363,75 +363,75 @@ and the derived files           conteq.ins,
 % \begin{macro}{\ConteqSetDefaultLayout}
 % The code to set the default layout.
 %    \begin{macrocode}
-\cs_new_protected:Nn \conteq_set_default_layout:n {
-  \tl_set:Nn \l_conteq_default_layout_tl {#1}
+\cs_new_protected:Nn \__conteq_set_default_layout:n {
+  \tl_set:Nn \l__conteq_default_layout_tl {#1}
 }
-\cs_new_eq:NN \ConteqSetDefaultLayout \conteq_set_default_layout:n
+\cs_new_eq:NN \ConteqSetDefaultLayout \__conteq_set_default_layout:n
 %    \end{macrocode}
 % \end{macro}
 %
 % \begin{macro}{\ConteqDefineLayout}
 % The code to define new layouts.
 %    \begin{macrocode}
-\cs_new_protected:Nn \conteq_define_layout:nnnnnnn {
-  \cs_new_protected:cn {conteq_#1_begin:} { #2 }
-  \cs_new_protected:cn {conteq_#1_lhs:n} { #3 }
-  \cs_new_protected:cn {conteq_#1_rhs_only:n} { #4 }
-  \cs_new_protected:cn {conteq_#1_rhs_expl:nn} { #5 }
-  \cs_new_protected:cn {conteq_#1_nl:} { #6 }
-  \cs_new_protected:cn {conteq_#1_end:} { #7 }
+\cs_new_protected:Nn \__conteq_define_layout:nnnnnnn {
+  \cs_new_protected:cn {__conteq_#1_begin:} { #2 }
+  \cs_new_protected:cn {__conteq_#1_lhs:n} { #3 }
+  \cs_new_protected:cn {__conteq_#1_rhs_only:n} { #4 }
+  \cs_new_protected:cn {__conteq_#1_rhs_expl:nn} { #5 }
+  \cs_new_protected:cn {__conteq_#1_nl:} { #6 }
+  \cs_new_protected:cn {__conteq_#1_end:} { #7 }
 }
-\cs_new_eq:NN \ConteqDefineLayout \conteq_define_layout:nnnnnnn
+\cs_new_eq:NN \ConteqDefineLayout \__conteq_define_layout:nnnnnnn
 %    \end{macrocode}
 % \end{macro}
 %
 % \subsection{Expansion utility function}
 % 
-% \begin{macro}{\conteq_args_once:Nn, \conteq_args_once:NV, \conteq_args_once:Nnn, \conteq_args_once:NVV}
+% \begin{macro}{\__conteq_args_once:Nn, \__conteq_args_once:NV, \__conteq_args_once:Nnn, \__conteq_args_once:NVV}
 % For ease of debugging(?) we construct a token list that contains exactly the tokens that a programmer would enter to create the layout manually. For that we need some fine-grained control over expansion.
 %
 %    \begin{macrocode}
-\cs_new:Npn \conteq_args_once:Nn #1#2 {
+\cs_new:Npn \__conteq_args_once:Nn #1#2 {
    \exp_not:o {#1 {#2}}
 }
-\cs_generate_variant:Nn \conteq_args_once:Nn {NV}
+\cs_generate_variant:Nn \__conteq_args_once:Nn {NV}
 
-\cs_new:Npn \conteq_args_once:Nnn #1#2#3 {
+\cs_new:Npn \__conteq_args_once:Nnn #1#2#3 {
    \exp_not:o {#1 {#2}{#3}}
 }
-\cs_generate_variant:Nn \conteq_args_once:Nnn {NVV}
+\cs_generate_variant:Nn \__conteq_args_once:Nnn {NVV}
 %    \end{macrocode}
 % \end{macro}
 %
 % \subsection{Main code}
 %
-% \begin{macro}{\conteq_print_line:nn}
+% \begin{macro}{\__conteq_print_line:nn}
 % This macro splits and prints one line of the table. The second argument is either |\\| or, for the last line of the table, empty.
 %    \begin{macrocode}
-\cs_new_protected:Npn \conteq_print_line:Nnn #1#2#3 {
-  \seq_set_split:Nnn \l_conteq_cells_seq { & } {#2}
-  \seq_pop_left:NN \l_conteq_cells_seq \l_conteq_rhs_tl
-  \tl_clear:N \l_conteq_expl_tl
-  \seq_if_empty:NF \l_conteq_cells_seq {
-    \seq_pop_left:NN \l_conteq_cells_seq \l_conteq_expl_tl
+\cs_new_protected:Npn \__conteq_print_line:Nnn #1#2#3 {
+  \seq_set_split:Nnn \l__conteq_cells_seq { & } {#2}
+  \seq_pop_left:NN \l__conteq_cells_seq \l__conteq_rhs_tl
+  \tl_clear:N \l__conteq_expl_tl
+  \seq_if_empty:NF \l__conteq_cells_seq {
+    \seq_pop_left:NN \l__conteq_cells_seq \l__conteq_expl_tl
   }
-  \tl_if_blank:VTF \l_conteq_expl_tl {
+  \tl_if_blank:VTF \l__conteq_expl_tl {
     \tl_put_right:Nx #1 {
-      \conteq_args_once:NV \conteq_rhs_only:n \l_conteq_rhs_tl
+      \__conteq_args_once:NV \__conteq_rhs_only:n \l__conteq_rhs_tl
     }
   }{
-    \tl_set:Nx \l_tmpa_tl {\exp_not:N \text {\exp_not:N \ConteqExplStyle {\exp_not:V \l_conteq_expl_tl}}}
+    \tl_set:Nx \l_tmpa_tl {\exp_not:N \text {\exp_not:N \ConteqExplStyle {\exp_not:V \l__conteq_expl_tl}}}
     \tl_put_right:Nx #1 {
-      \conteq_args_once:NVV \conteq_rhs_expl:nn \l_conteq_rhs_tl \l_tmpa_tl
+      \__conteq_args_once:NVV \__conteq_rhs_expl:nn \l__conteq_rhs_tl \l_tmpa_tl
     }
   }
-  \seq_if_empty:NF \l_conteq_cells_seq {
+  \seq_if_empty:NF \l__conteq_cells_seq {
     \msg_warning:nn{conteq}{ignoreddata}
   }
   \tl_put_right:Nn #1 {#3}
 }
-\cs_generate_variant:Nn \conteq_print_line:Nnn {Nno}
-\cs_generate_variant:Nn \conteq_print_line:Nnn {NVn}
+\cs_generate_variant:Nn \__conteq_print_line:Nnn {Nno}
+\cs_generate_variant:Nn \__conteq_print_line:Nnn {NVn}
 %    \end{macrocode}
 % \end{macro}
 
@@ -440,67 +440,67 @@ and the derived files           conteq.ins,
 %
 % The main environment of the package.
 %    \begin{macrocode}
-\NewEnviron{conteq}[1][\l_conteq_default_layout_tl]{
-  \tl_set:NV \l_conteq_body_tl \BODY
+\NewEnviron{conteq}[1][\l__conteq_default_layout_tl]{
+  \tl_set:NV \l__conteq_body_tl \BODY
   
-  \tl_if_blank:oT \l_conteq_body_tl{
+  \tl_if_blank:oT \l__conteq_body_tl{
       \msg_warning:nn{conteq}{empty}
   }
 %    \end{macrocode}
 
 % Figure out the layout to use\ldots
 %    \begin{macrocode}
-  \tl_set:Nn \l_conteq_layout_tl {#1}
+  \tl_set:Nn \l__conteq_layout_tl {#1}
 %    \end{macrocode}
 % and set the various functions accordingly, if the layout exists.
 %    \begin{macrocode}
-  \cs_if_exist:cTF {conteq_ \l_conteq_layout_tl _ begin:}{
+  \cs_if_exist:cTF {__conteq_ \l__conteq_layout_tl _ begin:}{
     \clist_map_inline:nn{begin:, lhs:n,rhs_only:n, rhs_expl:nn, nl:, end:}{
-      \cs_set_eq:cc {conteq_##1}{conteq_\l_conteq_layout_tl _##1}
+      \cs_set_eq:cc {__conteq_##1}{__conteq_\l__conteq_layout_tl _##1}
     }
   }{
-    \msg_critical:nnx{conteq}{undefined layout}{\l_conteq_layout_tl}
+    \msg_critical:nnx{conteq}{undefined layout}{\l__conteq_layout_tl}
   }
 %    \end{macrocode}
 
 % Split the body into individual lines.
 %    \begin{macrocode}
-  \seq_set_split:NnV \l_conteq_lines_seq { \\ } \l_conteq_body_tl
+  \seq_set_split:NnV \l__conteq_lines_seq { \\ } \l__conteq_body_tl
 
-  \tl_clear:N \l_conteq_result_tl
+  \tl_clear:N \l__conteq_result_tl
 %    \end{macrocode}
 
 % If there is only one line, simply print it.
 %    \begin{macrocode}
 
-  \int_case:nnn {\seq_count:N \l_conteq_lines_seq}
+  \int_case:nnn {\seq_count:N \l__conteq_lines_seq}
   {
     {1} {
-      \tl_put_right:Nx \l_conteq_result_tl {\exp_not:o \conteq_begin:}
-      \tl_put_right:NV \l_conteq_result_tl \l_conteq_body_tl
-      \tl_put_right:Nx \l_conteq_result_tl {\exp_not:o \conteq_end:}
+      \tl_put_right:Nx \l__conteq_result_tl {\exp_not:o \__conteq_begin:}
+      \tl_put_right:NV \l__conteq_result_tl \l__conteq_body_tl
+      \tl_put_right:Nx \l__conteq_result_tl {\exp_not:o \__conteq_end:}
     }
   }{
 %    \end{macrocode}
 
-% Otherwise extract the head and the last line, and print each line using |\conteq_print_line:nn|
+% Otherwise extract the head and the last line, and print each line using |\__conteq_print_line:nn|
 %    \begin{macrocode}
-    \seq_pop_left:NN \l_conteq_lines_seq \l_conteq_head_tl
-    \seq_pop_right:NN \l_conteq_lines_seq \l_conteq_lastline_tl
+    \seq_pop_left:NN \l__conteq_lines_seq \l__conteq_head_tl
+    \seq_pop_right:NN \l__conteq_lines_seq \l__conteq_lastline_tl
 
-    \tl_put_right:Nx \l_conteq_result_tl {\exp_not:o \conteq_begin:}
-    \tl_put_right:Nx \l_conteq_result_tl {
-      \conteq_args_once:NV \conteq_lhs:n \l_conteq_head_tl
+    \tl_put_right:Nx \l__conteq_result_tl {\exp_not:o \__conteq_begin:}
+    \tl_put_right:Nx \l__conteq_result_tl {
+      \__conteq_args_once:NV \__conteq_lhs:n \l__conteq_head_tl
     }
-    \seq_map_inline:Nn \l_conteq_lines_seq {
-      \conteq_print_line:Nno \l_conteq_result_tl {##1} {\conteq_nl:} 
+    \seq_map_inline:Nn \l__conteq_lines_seq {
+      \__conteq_print_line:Nno \l__conteq_result_tl {##1} {\__conteq_nl:} 
     }
-    \conteq_print_line:NVn \l_conteq_result_tl \l_conteq_lastline_tl {} 
-    \tl_put_right:Nx \l_conteq_result_tl {\exp_not:o \conteq_end:}
+    \__conteq_print_line:NVn \l__conteq_result_tl \l__conteq_lastline_tl {} 
+    \tl_put_right:Nx \l__conteq_result_tl {\exp_not:o \__conteq_end:}
   }
   % Use this for debugging
-  %\tl_show:N \l_conteq_result_tl
-  \tl_use:N \l_conteq_result_tl
+  %\tl_show:N \l__conteq_result_tl
+  \tl_use:N \l__conteq_result_tl
 }
 %    \end{macrocode}
 %\end{environment}