From 904150e75b6eed9874f5c3a54b90ac98acb9a059 Mon Sep 17 00:00:00 2001
From: Michael Smith <xmldoc@users.sourceforge.net>
Date: Tue, 26 Feb 2008 09:58:56 +0000
Subject: [PATCH] Added some templates for generating if-then-else conditional
 markup in groff, so let's use those instead of hard-coding it in multiple
 places...

---
 xsl/manpages/lists.xsl | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/xsl/manpages/lists.xsl b/xsl/manpages/lists.xsl
index 6ca5e2236..0d11d061d 100644
--- a/xsl/manpages/lists.xsl
+++ b/xsl/manpages/lists.xsl
@@ -132,7 +132,7 @@
   <!-- * TTY; so we do some fixed-width-font hackery with \h to make a -->
   <!-- * hanging indents (instead of using .IP, which has some -->
   <!-- * undesirable side effects under certain circumstances) -->
-  <xsl:text>.ie n \{\&#10;</xsl:text>
+  <xsl:call-template name="roff-if-else-start"/>
   <xsl:text>\h'-</xsl:text>
     <xsl:if test="not($list-indent = '')">
     <xsl:text>0</xsl:text>
@@ -147,17 +147,19 @@
   <xsl:text>'</xsl:text>
   </xsl:if>
   <xsl:apply-templates/>
-  <xsl:text>.\}&#10;</xsl:text>
   <!-- * else, we are not using for "nroff", but instead "troff" - which -->
   <!-- * means not for TTY, but for PS or whatever; so we’re not using a -->
   <!-- * fixed-width font, so use a real .IP instead -->
-  <xsl:text>.el \{\&#10;</xsl:text>
+  <xsl:call-template name="roff-else"/>
   <!-- * .IP generates a blank like of space, so let’s go backwards one -->
   <!-- * line up to compensate for that -->
   <xsl:text>.sp -1&#10;</xsl:text>
-  <!-- * the value 2 is the amount of indentation -->
-  <xsl:text>.IP \(bu 2&#10;</xsl:text>
+  <xsl:text>.IP \(bu 2.3&#10;</xsl:text>
+  <!-- * The value 2.3 is the amount of indentation; we use 2.3 instead -->
+  <!-- * of 2 because when the font family is New Century Schoolbook it -->
+  <!-- * seems to require the extra space. -->
   <xsl:apply-templates/>
+  <xsl:call-template name="roff-if-end"/>
   <xsl:text>.\}&#10;</xsl:text>
   <xsl:text>.RE&#10;</xsl:text>
 </xsl:template>
@@ -176,7 +178,7 @@
   <!-- * TTY; so we do some fixed-width-font hackery with \h to make a -->
   <!-- * hanging indents (instead of using .IP, which has some -->
   <!-- * undesirabel side effects under certain circumstances) -->
-  <xsl:text>.ie n \{\&#10;</xsl:text>
+  <xsl:call-template name="roff-if-else-start"/>
   <xsl:text>\h'-</xsl:text>
     <xsl:if test="not($list-indent = '')">
     <xsl:text>0</xsl:text>
@@ -194,11 +196,10 @@
   <xsl:text>'</xsl:text>
   </xsl:if>
   <xsl:apply-templates/>
-  <xsl:text>.\}&#10;</xsl:text>
   <!-- * else, we are not using for "nroff", but instead "troff" - which -->
   <!-- * means not for TTY, but for PS or whatever; so we’re not using a -->
   <!-- * fixed-width font, so use a real .IP instead -->
-  <xsl:text>.el \{\&#10;</xsl:text>
+  <xsl:call-template name="roff-else"/>
   <!-- * .IP generates a blank like of space, so let’s go backwards one -->
   <!-- * line up to compensate for that -->
   <xsl:text>.sp -1&#10;</xsl:text>
@@ -207,10 +208,12 @@
     <xsl:text>  </xsl:text>
   </xsl:if>
   <xsl:number format="1."/>
-  <xsl:text>" 4&#10;</xsl:text>
-  <!-- * the value 4 is the amount of indentation -->
+  <xsl:text>" 4.2&#10;</xsl:text>
+  <!-- * The value 4.2 is the amount of indentation; we use 4.2 instead -->
+  <!-- * of 4 because when the font family is Bookman it seems to require -->
+  <!-- * the extra space. -->
   <xsl:apply-templates/>
-  <xsl:text>.\}&#10;</xsl:text>
+  <xsl:call-template name="roff-if-end"/>
   <xsl:text>.RE&#10;</xsl:text>
   <xsl:text>&#10;</xsl:text>
 </xsl:template>
-- 
2.40.0