On Tue, Aug 17, 2021 at 07:21:27PM +0900, Akira Yokosawa wrote:
> Using \IfEbookSize or \IfSansSerif macro in the middle of a paragraph
> can cause extra white spaces around it.
> Fix this by adding the \ignorespaces command in their definitions.
>
> Also update the conditional text in styleguide.
>
> As the other conditional macros aren't used in this manner, they are
> not modified for the moment.
>
> Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> ---
> appendix/styleguide/styleguide.tex | 5 +++--
> perfbook-lt.tex | 6 ++++--
> 2 files changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/appendix/styleguide/styleguide.tex b/appendix/styleguide/styleguide.tex
> index 9bb9683f..24e891a3 100644
> --- a/appendix/styleguide/styleguide.tex
> +++ b/appendix/styleguide/styleguide.tex
> @@ -1013,8 +1013,9 @@ namely \qco{$-$}.\footnote{This rule assumes that math mode uses the
> same upright glyph as text mode.
> Our default font choice meets the assumption.
> \IfSansSerif{
> - One of the experimental targets ``1csf'' \emph{does} use a differnt font
> - for math mode figures as of October 2017.}{}
> + Experimental targets with sans-serif fonts such as
> + ``1csf'' and ``ebsf'' \emph{do} use a differnt font
I did s/differnt/different/ here. This looks like the sort of typo
that I would make! ;-)
Thanx, Paul
> + for math mode figures as of August 2021.}{}
> }
> For example,
>
> diff --git a/perfbook-lt.tex b/perfbook-lt.tex
> index 62959e04..b1c4f780 100644
> --- a/perfbook-lt.tex
> +++ b/perfbook-lt.tex
> @@ -117,13 +117,15 @@
> \newcommand{\IfHardCover}[2]{\ifthenelse{\boolean{hardcover}}{#1}{#2}}
> \newboolean{ebooksize}
> \setboolean{ebooksize}{false}
> -\newcommand{\IfEbookSize}[2]{\ifthenelse{\boolean{ebooksize}}{#1}{#2}}
> +\newcommand{\IfEbookSize}[2]{\ifthenelse{\boolean{ebooksize}}%
> + {\ignorespaces#1\ignorespaces}{\ignorespaces#2\ignorespaces}}
> \newboolean{afourpaper}
> \setboolean{afourpaper}{false}
> \newcommand{\IfAfourPaper}[2]{\ifthenelse{\boolean{afourpaper}}{#1}{#2}}
> \newboolean{sansserif}
> \setboolean{sansserif}{false}
> -\newcommand{\IfSansSerif}[2]{\ifthenelse{\boolean{sansserif}}{#1}{#2}}
> +\newcommand{\IfSansSerif}[2]{\ifthenelse{\boolean{sansserif}}%
> + {\ignorespaces#1\ignorespaces}{\ignorespaces#2\ignorespaces}}
> \newboolean{lmttforcode}
> \setboolean{lmttforcode}{true}
> \newcommand{\IfLmttForCode}[2]{\ifthenelse{\boolean{lmttforcode}}{#1}{#2}}
> --
> 2.17.1
>
>
