Switch public x-type interfaces to e-type

josephwright · Oct 13, 2023 · c97ebfb · c97ebfb
1 parent b730e03
commit c97ebfb
Show file tree

Hide file tree

Showing 10 changed files with 68 additions and 56 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,18 @@ Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to
 
 ## [Unreleased]
 
+### Deprecated
+- `x`-type variants
+ - `\siunitx_angle:x`
+ - `\siunitx_angle:xxx`
+ - `\siunitx_complex_number:x`
+ - `\siunitx_complex_quantity:xn`
+ - `\siunitx_print_number:x`
+ - `\siunitx_quantity_print:xV`
+ - `\siunitx_declare_prefix:Nnx`
+ - `\siunitx_declare_unit:Nx`
+ - `\siunitx_declare_unit:Nxn`
+
 ## [v3.3.7] - 2023-10-07
 
 ### Fixed

diff --git a/siunitx-angle.dtx b/siunitx-angle.dtx
@@ -64,8 +64,8 @@
 %
 % \begin{function}
 % {
-% \siunitx_angle:n, \siunitx_angle:x,
-% \siunitx_angle:nnn, \siunitx_angle:xxx
+% \siunitx_angle:n, \siunitx_angle:e,
+% \siunitx_angle:nnn, \siunitx_angle:eee
 % }
 % \begin{syntax}
 % \cs{siunitx_angle:n} \Arg{angle}
@@ -240,8 +240,8 @@
 % \end{macrocode}
 % \end{variable}
 %
-% \begin{macro}{\siunitx_angle:n, \siunitx_angle:x, \@@_angle:n}
-% \begin{macro}{\siunitx_angle:nnn, \siunitx_angle:xxx}
+% \begin{macro}{\siunitx_angle:n, \siunitx_angle:e, \@@_angle:n}
+% \begin{macro}{\siunitx_angle:nnn, \siunitx_angle:eee}
 % \begin{macro}{\@@_arc_convert:n, \@@_arc_convert:e}
 % The first step here is to force format conversion if required. Going to
 % a decimal is easy, going to arc format is a bit more painful: avoid
@@ -262,7 +262,7 @@
  }
  }
  }
-\cs_generate_variant:Nn \siunitx_angle:n { x }
+\cs_generate_variant:Nn \siunitx_angle:n { e , x }
 \cs_new_protected:Npn \@@_angle:n #1
  {
  \bool_if:NTF \l_@@_force_arc_bool
@@ -284,7 +284,7 @@
  {
  \bool_if:NTF \l_@@_force_decimal_bool
  {
- \siunitx_angle:x
+ \siunitx_angle:e
  { \fp_eval:n { #1 + (#2) / 60 + (#3) / 3600 } }
  }
  { \@@_arc_sign:nnn {#1} {#2} {#3} }
@@ -302,10 +302,10 @@
  \l_@@_seconds_tl
  }
  }
-\cs_generate_variant:Nn \siunitx_angle:nnn { xxx }
+\cs_generate_variant:Nn \siunitx_angle:nnn { eee , xxx }
 \cs_new_protected:Npn \@@_arc_convert:n #1
  {
- \siunitx_angle:xxx
+ \siunitx_angle:eee
  { \fp_eval:n { trunc(#1,0) } }
  { \fp_eval:n { trunc((#1 - trunc(#1,0)) * 60,0) } }
  {

diff --git a/siunitx-complex.dtx b/siunitx-complex.dtx
@@ -66,7 +66,7 @@
 % assumptions concerning \LaTeXe{} commands in the submodule beyond those in the
 % core number and unit submodules.
 %
-% \begin{function}{\siunitx_complex_number:n, \siunitx_complex_number:x}
+% \begin{function}{\siunitx_complex_number:n, \siunitx_complex_number:e}
 % \begin{syntax}
 % \cs{siunitx_complex_number:n} \Arg{number}
 % \end{syntax}
@@ -96,7 +96,7 @@
 % \begin{function}
 % {
 % \siunitx_complex_quantity:nn ,
-% \siunitx_complex_quantity:xn ,
+% \siunitx_complex_quantity:en ,
 % \siunitx_complex_quantity:nnn
 % }
 % \begin{syntax}
@@ -691,9 +691,9 @@
 % \end{macrocode}
 % \end{variable}
 %
-% \begin{macro}{\siunitx_complex_number:n, \siunitx_complex_number:x}
+% \begin{macro}{\siunitx_complex_number:n, \siunitx_complex_number:e}
 % \begin{macro}{\siunitx_complex_number:nn, \@@_number:nn}
-% \begin{macro}{\siunitx_complex_quantity:nn, \siunitx_complex_quantity:xn}
+% \begin{macro}{\siunitx_complex_quantity:nn, \siunitx_complex_quantity:en}
 % \begin{macro}{\siunitx_complex_quantity:nnn, \@@_quantity:nnn}
 % The work here is pretty trivial: only conversion between forms makes
 % things a bit more intricate.
@@ -722,7 +722,7 @@
  }
  }
  }
-\cs_generate_variant:Nn \siunitx_complex_number:n { x }
+\cs_generate_variant:Nn \siunitx_complex_number:n { e, x }
 \cs_new_protected:Npn \siunitx_complex_number:nn #1#2
  {
  \bool_lazy_or:nnTF
@@ -740,7 +740,7 @@
  { \l_@@_force_cartesian_bool }
  { \l_siunitx_number_parse_bool }
  {
- \siunitx_complex_number:x
+ \siunitx_complex_number:e
  { \@@_convert_cartesian:nn {#1} {#2} }
  }
  {
@@ -770,7 +770,7 @@
  { \siunitx_quantity:nn {#1} {#2} }
  }
  }
-\cs_generate_variant:Nn \siunitx_complex_quantity:nn { x }
+\cs_generate_variant:Nn \siunitx_complex_quantity:nn { e , x }
 \cs_new_protected:Npn \siunitx_complex_quantity:nnn #1#2#3
  {
  \bool_lazy_or:nnTF
@@ -788,7 +788,7 @@
  { \l_@@_force_cartesian_bool }
  { \l_siunitx_number_parse_bool }
  {
- \siunitx_complex_quantity:xn
+ \siunitx_complex_quantity:en
  { \@@_convert_cartesian:nn {#1} {#2} }
  {#3}
  }

diff --git a/siunitx-compound.dtx b/siunitx-compound.dtx
@@ -546,7 +546,7 @@
  \group_begin:
  \bool_set_false:N \l_@@_unit_bool
  \@@_format:nn {#1} { }
- \@@_print:N \siunitx_print_number:x
+ \@@_print:N \siunitx_print_number:e
  \group_end:
  }
 \cs_new_protected:Npn \@@_format:nn #1#2
@@ -862,10 +862,10 @@
  { \tl_if_empty_p:N \l_@@_exp_tl }
  {
  \siunitx_print_number:V \l_@@_bracket_open_tl
- \@@_print:N \siunitx_print_number:x
+ \@@_print:N \siunitx_print_number:e
  \siunitx_print_number:V \l_@@_bracket_close_tl
  }
- { \@@_print:N \siunitx_print_number:x }
+ { \@@_print:N \siunitx_print_number:e }
  \@@_print_quantity:n { \c_empty_tl }
  }
  }

diff --git a/siunitx-emulation.dtx b/siunitx-emulation.dtx
@@ -1050,7 +1050,7 @@
  {
  \quark_if_recursion_tail_stop:N #1
  \bool_new:c { g_@@_unit_warning_ \token_to_str:N #1 _bool }
- \siunitx_declare_unit:Nx #1
+ \siunitx_declare_unit:Ne #1
  {
  \exp_not:N \bool_if:NF
  \exp_not:c { g_@@_unit_warning_ \token_to_str:N #1 _bool }

diff --git a/siunitx-print.dtx b/siunitx-print.dtx
@@ -105,9 +105,9 @@
 % \begin{function}
 % {
 % \siunitx_print_number:n, \siunitx_print_number:V,
-% \siunitx_print_number:x,
-% \siunitx_print_unit:n, \siunitx_print_unit:V, \siunitx_print_unit:o,
-% \siunitx_print_unit:x
+% \siunitx_print_number:e,
+% \siunitx_print_unit:n, \siunitx_print_unit:V, \siunitx_print_unit:e,
+% \siunitx_print_unit:o
 % }
 % \begin{syntax}
 % \cs{siunitx_print_number:n} \Arg{material}
@@ -458,9 +458,9 @@
 % \begin{macro}
 % {
 % \siunitx_print_number:n, \siunitx_print_number:V,
-% \siunitx_print_number:x,
-% \siunitx_print_unit:n, \siunitx_print_unit:V, \siunitx_print_unit:o,
-% \siunitx_print_unit:x
+% \siunitx_print_number:e,
+% \siunitx_print_unit:n, \siunitx_print_unit:V, \siunitx_print_unit:e,
+% \siunitx_print_unit:o
 % }
 % \begin{macro}{\@@_aux:nn}
 % The main printing function doesn't actually need to do very much: just set
@@ -471,10 +471,10 @@
 % \begin{macrocode}
 \cs_new_protected:Npn \siunitx_print_number:n #1
  { \@@_aux:nn { number } {#1} }
-\cs_generate_variant:Nn \siunitx_print_number:n { V , x }
+\cs_generate_variant:Nn \siunitx_print_number:n { V , e , x }
 \cs_new_protected:Npn \siunitx_print_unit:n #1
  { \@@_aux:nn { unit } {#1} }
-\cs_generate_variant:Nn \siunitx_print_unit:n { V , o , x }
+\cs_generate_variant:Nn \siunitx_print_unit:n { V , e , o , x }
 \cs_new_protected:Npn \@@_aux:nn #1#2
  {
  \tl_if_empty:oF {#2}

diff --git a/siunitx-quantity.dtx b/siunitx-quantity.dtx
@@ -78,7 +78,7 @@
 % \begin{function}
 % {
 % \siunitx_quantity_print:nn, \siunitx_quantity_print:nV,
-% \siunitx_quantity_print:VV, \siunitx_quantity_print:xV
+% \siunitx_quantity_print:VV, \siunitx_quantity_print:eV
 % }
 % \begin{syntax}
 % \cs{siunitx_quantity_print:nn} \Arg{number} \Arg{unit}
@@ -311,7 +311,7 @@
  {
  \bool_if:NTF \l_@@_uncert_bracket_bool
  {
- \siunitx_quantity_print:xV
+ \siunitx_quantity_print:eV
  {
  \exp_not:n {#2}
  \exp_not:V \l_@@_bracket_open_tl
@@ -376,7 +376,7 @@
 % \begin{macro}
 % {
 % \siunitx_quantity_print:nn, \siunitx_quantity_print:nV,
-% \siunitx_quantity_print:VV, \siunitx_quantity_print:xV
+% \siunitx_quantity_print:VV, \siunitx_quantity_print:eV
 % }
 % For printing a single part of a quantity. This is needed for compound
 % quantities and so is public: that's also the reason for passing both
@@ -403,7 +403,7 @@
  { \siunitx_print_unit:o { \use_none:n #2 } }
  }
  }
-\cs_generate_variant:Nn \siunitx_quantity_print:nn { nV , VV , xV }
+\cs_generate_variant:Nn \siunitx_quantity_print:nn { nV , VV , eV , xV }
 % \end{macrocode}
 % \end{macro}
 %
@@ -437,13 +437,13 @@
 % in quantities. This is done here as it avoids a dependency in
 % \pkg{siunitx-unit} on options it does not contain.
 % \begin{macrocode}
-\siunitx_declare_unit:Nxn \arcminute
+\siunitx_declare_unit:Nen \arcminute
  { \@@_non_latin:n { "02B9 } }
  { quantity-product = { } }
-\siunitx_declare_unit:Nxn \arcsecond
+\siunitx_declare_unit:Nen \arcsecond
  { \@@_non_latin:n { "02BA } }
  { quantity-product = { } }
-\siunitx_declare_unit:Nxn \degree
+\siunitx_declare_unit:Nen \degree
  { \@@_non_latin:n { "00B0 } }
  { quantity-product = { } }
 % \end{macrocode}

diff --git a/siunitx-symbol.dtx b/siunitx-symbol.dtx
@@ -159,7 +159,7 @@
 % \begin{macrocode}
  \@@_if_replace:NnT \degree { "00B0 }
  {
- \siunitx_declare_unit:Nxn \degree
+ \siunitx_declare_unit:Nen \degree
  {
  \exp_not:N \ifmmode
  \@ifpackageloaded { unicode-math }
@@ -189,7 +189,7 @@
  \str_if_eq:VVTF \l_@@_tmpa_tl \l_@@_tmpb_tl
  {
  \group_end:
- \siunitx_declare_unit:Nx \degreeCelsius
+ \siunitx_declare_unit:Ne \degreeCelsius
  {
  \exp_not:N \ifmmode
  \@ifpackageloaded { unicode-math }
@@ -225,7 +225,7 @@
  \exp_not:N \text
  { \exp_not:N \ensuremath { \exp_not:V \l_@@_tmpa_tl } }
  }
- \siunitx_declare_unit:Nx \ohm
+ \siunitx_declare_unit:Ne \ohm
  {
  \@ifpackageloaded { fourier }
  {
@@ -260,7 +260,7 @@
 % \begin{macrocode}
  \@@_if_replace:NnT \micro { "03BC }
  {
- \siunitx_declare_prefix:Nnx \micro { -6 }
+ \siunitx_declare_prefix:Nne \micro { -6 }
  {
  \exp_not:N \text
  {

diff --git a/siunitx-table.dtx b/siunitx-table.dtx
@@ -1263,7 +1263,7 @@
  { \siunitx_print_number:n { #1#2#3 } }
  \hbox_set:Nn \l_@@_decimal_box
  {
- \siunitx_print_number:x
+ \siunitx_print_number:e
  { \@@_print_marker_aux:w #4 \q_stop }
  }
  \@@_center_marker: