On Tue, 17 Aug 2021 11:29:51 -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! ;-)
So, I made the typo way back in 2017.
A typo in a footnote in Style Guide is unlikely to be caught. ;-)
Thanks, Akira
>
> 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
>>
>>
