From 6750538455a5bfdbaff3635233c227670ec9b27f Mon Sep 17 00:00:00 2001
From: Bill Spitzak <spitzak@gmail.com>
Date: Fri, 2 Jan 2015 18:29:22 -0800
Subject: [PATCH] doc: remove redundant subtitles

Put the argument lists next to the event/message title, which I
think makes it a lot easier to understand, and remove redundant
"values" title from enumerations.

Reviewed-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
---
 doc/publican/protocol-to-docbook.xsl | 38 ++++++++++++++++++----------
 1 file changed, 25 insertions(+), 13 deletions(-)

diff --git a/doc/publican/protocol-to-docbook.xsl b/doc/publican/protocol-to-docbook.xsl
index ca1308f1..a886b7f9 100644
--- a/doc/publican/protocol-to-docbook.xsl
+++ b/doc/publican/protocol-to-docbook.xsl
@@ -147,7 +147,27 @@
 </xsl:template>
 
 <!-- Request/event list -->
-<xsl:template match="request|event|enum">
+<xsl:template match="request|event">
+  <section id="protocol-spec-{../@name}-{name()}-{@name}">
+    <title>
+      <xsl:value-of select="../@name"/>::<xsl:value-of select="@name" />
+      <xsl:if test="description/@summary">
+        - <xsl:value-of select="description/@summary" />
+      </xsl:if>
+    </title>
+    <para>
+      <variablelist>
+        <xsl:apply-templates select="arg"/>
+      </variablelist>
+    </para>
+    <xsl:call-template name="break">
+      <xsl:with-param name="text" select="description" />
+    </xsl:call-template>
+  </section>
+</xsl:template>
+
+<!-- Enumeration -->
+<xsl:template match="enum">
   <section id="protocol-spec-{../@name}-{name()}-{@name}">
     <title>
       <xsl:value-of select="../@name"/>::<xsl:value-of select="@name" />
@@ -158,20 +178,12 @@
     <xsl:call-template name="break">
       <xsl:with-param name="text" select="description" />
     </xsl:call-template>
-    <xsl:if test="arg">
-      <variablelist>
-        <title><xsl:value-of select="../@name"/>::<xsl:value-of select="@name" /> arguments</title>
-        <xsl:apply-templates select="arg"/>
-      </variablelist>
-    </xsl:if>
-    <xsl:if test="entry">
-      <variablelist>
-        <title><xsl:value-of select="../@name"/>::<xsl:value-of select="@name" /> values</title>
-          <xsl:apply-templates select="entry"/>
-      </variablelist>
-    </xsl:if>
+    <variablelist>
+      <xsl:apply-templates select="entry"/>
+    </variablelist>
   </section>
 </xsl:template>
+
 </xsl:stylesheet>
 
 <!-- vim: set expandtab shiftwidth=2: -->