[PATCH 0/3] Tweak section-level epigraph

[PATCH 0/3] Tweak section-level epigraph

[Akira Yokosawa]

From 40c5e61208be1df272b99fa103b9d74ddbb1c379 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sat, 1 Sep 2018 08:45:12 +0900
Subject: [PATCH 0/3] Tweak section-level epigraph

Hi Paul,

Recently, you added section-level epigraphs in debugging chapter.
I noticed minor issues in there appearances.

  1. 1st paragraph of a section is now indented.
  2. In "1csf" target, section-level epigraph of sans serif font
     looks inconsistent with chapter-level epigraph of roman font.

Indent right after an epigraph can be suppressed by using
"noindentafter" package. However, Ubuntu Trusty has a buggy one.
So patch #1 imports the latest one.
Patch #2 enables the \NoIndentAfterCmd setting for \epigraph{}.
Patch #3 removes font family setting in \Epigraph{} for chapter-
level epigraph.

Note that Patch #1 contains trailing blanks.

         Thanks, Akira

Akira Yokosawa (3):
  Import noindentafter.sty 0.2.2
  Suppress indent right after section-level epigraph
  Remove font family specifier in \Epigraph{}

 Makefile          |   1 +
 noindentafter.sty | 189 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
 perfbook.tex      |   4 +-
 3 files changed, 193 insertions(+), 1 deletion(-)
 create mode 100644 noindentafter.sty

-- 
2.7.4

[PATCH 1/3] Import noindentafter.sty 0.2.2

[Akira Yokosawa]

From 40ceedb7e148dddaed13ece5ab94ad409101bdde Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Fri, 31 Aug 2018 20:16:23 +0900
Subject: [PATCH 1/3] Import noindentafter.sty 0.2.2

For those using TeX Live 2013/Debian on Ubuntu Trusty which has a
buggy version of noindentafter.sty (0.0.1), import the latest one.

"noindentafter" can be used to suppress indent right after section-
level epigraphs added in commit 87181d9e5d6a ("debugging: Add
section-level epigraphs").

NOTE: Trailing blanks are kept as they are.

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 noindentafter.sty | 189 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 189 insertions(+)
 create mode 100644 noindentafter.sty

diff --git a/noindentafter.sty b/noindentafter.sty
new file mode 100644
index 0000000..2ea6e22
--- /dev/null
+++ b/noindentafter.sty
@@ -0,0 +1,189 @@
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% \iffalse %%%%
+%                                                                              %
+%  Copyright (c) 2014 - Michiel Helvensteijn - www.mhelvens.net                %
+%                                                                              %
+%  http://latex-noindentafter.googlecode.com                                   %
+%                                                                              %
+%  This work may be distributed and/or modified under the conditions           %
+%  of the LaTeX Project Public License, either version 1.3 of this             %
+%  license or (at your option) any later version. The latest version           %
+%  of this license is in     http://www.latex-project.org/lppl.txt             %
+%  and version 1.3 or later is part of all distributions of LaTeX              %
+%  version 2005/12/01 or later.                                                %
+%                                                                              %
+%  This work has the LPPL maintenance status `maintained'.                     %
+%                                                                              %
+%  The Current Maintainer of this work is Michiel Helvensteijn.                %
+%                                                                              %
+%  This work consists of the files noindentafter.tex and noindentafter.sty.    %
+%                                                                              %
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% \fi %%%%
+
+% \CheckSum{50}
+%
+% \CharacterTable
+%  {Upper-case    \A\B\C\D\E\F\G\H\I\J\K\L\M\N\O\P\Q\R\S\T\U\V\W\X\Y\Z
+%   Lower-case    \a\b\c\d\e\f\g\h\i\j\k\l\m\n\o\p\q\r\s\t\u\v\w\x\y\z
+%   Digits        \0\1\2\3\4\5\6\7\8\9
+%   Exclamation   \!     Double quote  \"     Hash (number) \#
+%   Dollar        \$     Percent       \%     Ampersand     \&
+%   Acute accent  \'     Left paren    \(     Right paren   \)
+%   Asterisk      \*     Plus          \+     Comma         \,
+%   Minus         \-     Point         \.     Solidus       \/
+%   Colon         \:     Semicolon     \;     Less than     \<
+%   Equals        \=     Greater than  \>     Question mark \?
+%   Commercial at \@     Left bracket  \[     Backslash     \\
+%   Right bracket \]     Circumflex    \^     Underscore    \_
+%   Grave accent  \`     Left brace    \{     Vertical bar  \|
+%   Right brace   \}     Tilde         \~}
+
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+% \subsection{Package Info}                                                    %
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%
+%    \begin{macrocode}
+\NeedsTeXFormat{LaTeX2e}
+\ProvidesPackage{noindentafter}[2014/11/30 0.2.2
+  prevent paragraph indentation after specific environments or macros]
+%    \end{macrocode}
+%
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+% \subsection{Packages}                                                        %
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%
+%    \begin{macrocode}
+\RequirePackage{etoolbox}
+%    \end{macrocode}
+%
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+% \subsection{Patches}                                                         %
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+  %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%%% \needspace{5\baselineskip}\begin{macro}{\end}
+%
+%  The package |etoolbox| provides the command
+%  |\AfterEndEnvironment| which creates a hook executed at a
+%  very late point inside the |\end| command. However, this
+%  hook is still located before |\ignorespaces|, which is
+%  too early to properly suppress the indention after an
+%  environment. Therefore another hook is now added to |\end|
+%  using |\patchcmd|. This new hook puts new code at the very
+%  end.
+%
+%    \begin{macrocode}
+\patchcmd\end{%
+  \if@ignore\@ignorefalse\ignorespaces\fi%
+}{%
+  \if@ignore\@ignorefalse\ignorespaces\fi%
+  \csuse{@noindent@#1@hook}%
+}{}{%
+  \PackageWarningNoLine{noindentafter}{%
+    Patching `\string\end' failed!\MessageBreak%
+    `\string\NoIndentAfter...' commands won't work%
+  }%
+}
+%    \end{macrocode}
+%
+%\end{macro}%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+% \subsection{Macros}                                                          %
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+  %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%%% \needspace{5\baselineskip}\begin{macro}{\@NoIndentAfter}
+%
+%  \noindent This command implements the main principle
+%  behind this package. It checks whether it is followed by
+%  a paragraph. If so, the command |\par| is temporarily
+%  changed using |\everypar|, so that the following paragraph
+%  is not indented. Immediately afterwards, default paragraph
+%  behavior is restored with |\@restorepar| (from the \LaTeX{}
+%  base).
+%
+%    \begin{macrocode}
+\newcommand*\@NoIndentAfter{%
+  \@ifnextchar\par{%
+    \def\par{%
+      \everypar{\setbox\z@\lastbox\everypar{}}%
+      \@restorepar%
+    }%
+  }{}%
+}
+%    \end{macrocode}
+%
+%\end{macro}%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+  %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%%% \needspace{5\baselineskip}\begin{macro}{\NoIndentAfterThis}
+%
+%  \noindent Enforce a paragraph break and suppress
+%  indentation for whatever follows.
+% 
+%    \begin{macrocode}
+\newrobustcmd*{\NoIndentAfterThis}{\@NoIndentAfter\par\par}
+%    \end{macrocode}
+% 
+%\end{macro}%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+  %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+  % \needspace{5\baselineskip}\begin{macro}{\NoIndentAfterEnv}
+%%% \marg{environment}\\
+%
+%  \noindent Append |\@NoIndentAfter| to the output of
+%  \meta{environment} by using the new environment hook.
+% 
+%    \begin{macrocode}
+\newrobustcmd{\NoIndentAfterEnv}[1]{%
+  \csdef{@noindent@#1@hook}{\@NoIndentAfter}%
+}
+%    \end{macrocode}
+% 
+%\end{macro}%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+  %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+  % \needspace{5\baselineskip}\begin{macro}{\NoIndentAfterCmd}
+%%% \marg{command}\\
+%
+%  \noindent Append |\NoIndentAfterThis| to the output of
+%  \meta{command}.
+% 
+%    \begin{macrocode}
+\newrobustcmd*{\NoIndentAfterCmd}[1]{%
+  \apptocmd{#1}{\NoIndentAfterThis}{}{%
+    \PackageWarning{noindentafter}{%
+      Patching `\string#1' failed!\MessageBreak%
+      `\string\NoIndentAfterCmd' won't work%
+    }%
+  }%
+}
+%    \end{macrocode}
+% 
+%\end{macro}%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+
+
+
-- 
2.7.4

[PATCH 2/3] Suppress indent right after section-level epigraph

[Akira Yokosawa]

From 80b06ac34477b84041fa5757d1c9755e9a7b8a47 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Fri, 31 Aug 2018 20:27:39 +0900
Subject: [PATCH 2/3] Suppress indent right after section-level epigraph

By the help of "noindentafter" package, indent right after the
section level epigraphs can be suppressed.

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 Makefile     | 1 +
 perfbook.tex | 2 ++
 2 files changed, 3 insertions(+)

diff --git a/Makefile b/Makefile
index 3e9e2c2..a88553b 100644
--- a/Makefile
+++ b/Makefile
@@ -3,6 +3,7 @@ LATEXSOURCES = \
 	legal.tex \
 	glossary.tex \
 	qqz.sty origpub.sty \
+	noindentafter.sty \
 	pfbook.cls \
 	ushyphex.tex pfhyphex.tex \
 	*/*.tex \
diff --git a/perfbook.tex b/perfbook.tex
index e58d963..677c354 100644
--- a/perfbook.tex
+++ b/perfbook.tex
@@ -61,6 +61,8 @@
 \usepackage{siunitx} % for \num{} macro
 \sisetup{group-minimum-digits=4,group-separator={,},group-digits=integer}
 \usepackage{multirow}
+\usepackage{noindentafter}
+\NoIndentAfterCmd{\epigraph}

 % custom packages

-- 
2.7.4

[PATCH 3/3] Remove font family specifier in \Epigraph{}

[Akira Yokosawa]

From 40c5e61208be1df272b99fa103b9d74ddbb1c379 Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Fri, 31 Aug 2018 20:48:07 +0900
Subject: [PATCH 3/3] Remove font family specifier in \Epigraph{}

Now that section-level epigraphs are added, roman font of chapter-
level epigraphs looks inconsistent in sans serif font target.
Let's remove the font specifier in the definition of \Epigraph{}.

NOTE: This is a partial revert of commit 649eb172512a ("Makefile:
Add target '1csf' (one-column sans serif)").

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 perfbook.tex | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/perfbook.tex b/perfbook.tex
index 677c354..fb35c5a 100644
--- a/perfbook.tex
+++ b/perfbook.tex
@@ -187,7 +187,7 @@
 %\newcommand{\mytexttrademark}{\textsuperscript\texttrademark}
 %\newcommand{\mytextregistered}{\textsuperscript\textregistered}

-\newcommand{\Epigraph}[2]{\epigraphhead[65]{\rmfamily\epigraph{#1}{#2}}}
+\newcommand{\Epigraph}[2]{\epigraphhead[65]{\epigraph{#1}{#2}}}

 \input{ushyphex} % Hyphenation exceptions for US English from hyphenex package
 \input{pfhyphex} % Hyphenation exceptions for perfbook
-- 
2.7.4

Re: [PATCH 0/3] Tweak section-level epigraph

[Paul E. McKenney]

On Sat, Sep 01, 2018 at 09:11:36AM +0900, Akira Yokosawa wrote:
> >From 40c5e61208be1df272b99fa103b9d74ddbb1c379 Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Sat, 1 Sep 2018 08:45:12 +0900
> Subject: [PATCH 0/3] Tweak section-level epigraph
> 
> Hi Paul,
> 
> Recently, you added section-level epigraphs in debugging chapter.
> I noticed minor issues in there appearances.
> 
>   1. 1st paragraph of a section is now indented.
>   2. In "1csf" target, section-level epigraph of sans serif font
>      looks inconsistent with chapter-level epigraph of roman font.
> 
> Indent right after an epigraph can be suppressed by using
> "noindentafter" package. However, Ubuntu Trusty has a buggy one.
> So patch #1 imports the latest one.
> Patch #2 enables the \NoIndentAfterCmd setting for \epigraph{}.
> Patch #3 removes font family setting in \Epigraph{} for chapter-
> level epigraph.
> 
> Note that Patch #1 contains trailing blanks.

Nice!!!

But given the license and this line from noindentafter.sty, don't
we either have to also add noindentafter.tex on the one hand or
say where to get them on the other?

"This work consists of the files noindentafter.tex and noindentafter.sty."

Or am I misinterpreting the license?

							Thanx, Paul

>          Thanks, Akira
> 
> Akira Yokosawa (3):
>   Import noindentafter.sty 0.2.2
>   Suppress indent right after section-level epigraph
>   Remove font family specifier in \Epigraph{}
> 
>  Makefile          |   1 +
>  noindentafter.sty | 189 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  perfbook.tex      |   4 +-
>  3 files changed, 193 insertions(+), 1 deletion(-)
>  create mode 100644 noindentafter.sty
> 
> -- 
> 2.7.4
> 

Re: [PATCH 0/3] Tweak section-level epigraph

[Akira Yokosawa]

On 2018/09/01 10:29:47 -0700, Paul E. McKenney wrote:
> On Sat, Sep 01, 2018 at 09:11:36AM +0900, Akira Yokosawa wrote:
>> >From 40c5e61208be1df272b99fa103b9d74ddbb1c379 Mon Sep 17 00:00:00 2001
>> From: Akira Yokosawa <akiyks@gmail.com>
>> Date: Sat, 1 Sep 2018 08:45:12 +0900
>> Subject: [PATCH 0/3] Tweak section-level epigraph
>>
>> Hi Paul,
>>
>> Recently, you added section-level epigraphs in debugging chapter.
>> I noticed minor issues in there appearances.
>>
>>   1. 1st paragraph of a section is now indented.
>>   2. In "1csf" target, section-level epigraph of sans serif font
>>      looks inconsistent with chapter-level epigraph of roman font.
>>
>> Indent right after an epigraph can be suppressed by using
>> "noindentafter" package. However, Ubuntu Trusty has a buggy one.
>> So patch #1 imports the latest one.
>> Patch #2 enables the \NoIndentAfterCmd setting for \epigraph{}.
>> Patch #3 removes font family setting in \Epigraph{} for chapter-
>> level epigraph.
>>
>> Note that Patch #1 contains trailing blanks.
> 
> Nice!!!
> 
> But given the license and this line from noindentafter.sty, don't
> we either have to also add noindentafter.tex on the one hand or
> say where to get them on the other?
> 
> "This work consists of the files noindentafter.tex and noindentafter.sty."
> 
> Or am I misinterpreting the license?
> 
You are right!
But the actual noindentafter 0.2.2 has more files necessary to
compile noindentafter.tex.
So, I'll add the info where to get the whole package.
A patch with a proper commit message is coming soon.

        Thanks, Akira
> 							Thanx, Paul
> 
>>          Thanks, Akira
>>
>> Akira Yokosawa (3):
>>   Import noindentafter.sty 0.2.2
>>   Suppress indent right after section-level epigraph
>>   Remove font family specifier in \Epigraph{}
>>
>>  Makefile          |   1 +
>>  noindentafter.sty | 189 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  perfbook.tex      |   4 +-
>>  3 files changed, 193 insertions(+), 1 deletion(-)
>>  create mode 100644 noindentafter.sty
>>
>> -- 
>> 2.7.4
>>
> 

[PATCH] Add info where to obtain 'noindentafter' package

[Akira Yokosawa]

From bb7727a5e3a9a22ec53a870f09b7bbf39448f95e Mon Sep 17 00:00:00 2001
From: Akira Yokosawa <akiyks@gmail.com>
Date: Sun, 2 Sep 2018 08:12:36 +0900
Subject: [PATCH] Add info where to obtain 'noindentafter' package

LPPL 1.3c Clause 6.d.2 requires us to provide information to obtain
a complete, unmodified copy of the Work.

Reported-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
 noindentafter.sty | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/noindentafter.sty b/noindentafter.sty
index 2ea6e22..7a36224 100644
--- a/noindentafter.sty
+++ b/noindentafter.sty
@@ -1,3 +1,8 @@
+%% Notice to comply with Clause 6 of LPPL v 1.3c
+%% This file is copied from TeX Live 2015/Debian.
+%% Whole package including documentation can be obtained from:
+%%     https://ctan.org/pkg/noindentafter
+
 %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% \iffalse %%%%
 %                                                                              %
 %  Copyright (c) 2014 - Michiel Helvensteijn - www.mhelvens.net                %
-- 
2.7.4

Re: [PATCH] Add info where to obtain 'noindentafter' package

[Paul E. McKenney]

On Sun, Sep 02, 2018 at 08:33:04AM +0900, Akira Yokosawa wrote:
> >From bb7727a5e3a9a22ec53a870f09b7bbf39448f95e Mon Sep 17 00:00:00 2001
> From: Akira Yokosawa <akiyks@gmail.com>
> Date: Sun, 2 Sep 2018 08:12:36 +0900
> Subject: [PATCH] Add info where to obtain 'noindentafter' package
> 
> LPPL 1.3c Clause 6.d.2 requires us to provide information to obtain
> a complete, unmodified copy of the Work.
> 
> Reported-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>

Very good, applied and all four pushed, thank you!

							Thanx, Paul

> ---
>  noindentafter.sty | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/noindentafter.sty b/noindentafter.sty
> index 2ea6e22..7a36224 100644
> --- a/noindentafter.sty
> +++ b/noindentafter.sty
> @@ -1,3 +1,8 @@
> +%% Notice to comply with Clause 6 of LPPL v 1.3c
> +%% This file is copied from TeX Live 2015/Debian.
> +%% Whole package including documentation can be obtained from:
> +%%     https://ctan.org/pkg/noindentafter
> +
>  %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% \iffalse %%%%
>  %                                                                              %
>  %  Copyright (c) 2014 - Michiel Helvensteijn - www.mhelvens.net                %
> -- 
> 2.7.4
> 
>