On Tue, Aug 17, 2021 at 11:29:51AM -0700, Paul E. McKenney wrote:
> 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! ;-)
Other than that, all queued and pushed, thank you!
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
> >
> >
