From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-32494-mason-zsh=primenet.com.au@zsh.org>
Received: (qmail 23908 invoked by alias); 17 Mar 2014 16:02:16 -0000
Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
X-No-Archive: yes
List-Id: Zsh Workers List <zsh-workers.zsh.org>
List-Post: <mailto:zsh-workers@zsh.org>
List-Help: <mailto:zsh-workers-help@zsh.org>
X-Seq: 32494
Received: (qmail 2237 invoked from network); 17 Mar 2014 16:02:10 -0000
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on f.primenet.com.au
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE
	autolearn=ham version=3.3.2
X-Biglobe-Sender: <takimoto-j@kba.biglobe.ne.jp>
From: "Jun T." <takimoto-j@kba.biglobe.ne.jp>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: quoted-printable
Subject: A possible fix of an indentation problem in manages
Message-Id: <10AF07DF-B78A-4EDD-8DD2-4E0766CA8E96@kba.biglobe.ne.jp>
Date: Tue, 18 Mar 2014 01:01:32 +0900
To: zsh-workers@zsh.org
Mime-Version: 1.0 (Mac OS X Mail 7.2 \(1874\))
X-Mailer: Apple Mail (2.1874)

This is a reply to old posts X-seq: 29369, 29370
http://www.zsh.org/mla/workers/2011/msg00804.html
http://www.zsh.org/mla/workers/2011/msg00805.html

As pointed out in the above post, indentation of the text in the
description of _arguments in zshcompsys.1 has a problem. I think
that the place where the problem starts is:

                            ....
                            the  first  optarg  that  follows the =
optspec; see
                            below.

              It is possible for options with a literal `+' or `=3D' to  =
appear,
              but that character must be quoted, for example `-\+'.=20


Here, the indentation level is reduced by 2, but I think it should
be reduce only by 1.=20
This is due to that ENDITEM is redefined to emit '.RE' and used twice
by the macros item() and enditem().

A similar problem exists also in zshzle.1 (description of bindkey)
and zshmodule.1 (zparseopt).

The following patch *seems* to fix the problem, at least for the current
manpages. But even if it works, there may be a better way than to
re-re-define ENDITEM...


diff --git a/Doc/zman.yo b/Doc/zman.yo
index 8d17f78..3329cb9 100644
--- a/Doc/zman.yo
+++ b/Doc/zman.yo
@@ -110,7 +110,7 @@ def(startitem)(0)(\
 def(enditem)(0)(\
   ENDITEM()\
   STDPAR()\
-  +redef(ENDITEM)(0)(NOTRANS(.RE)STDPAR())\
+  +redef(ENDITEM)(0)(NOTRANS(.RE)STDPAR()redef(ENDITEM)(0)(CMT()))\
 )
 def(item)(2)(\
   NOTRANS(.TP)+NL()\
@@ -121,7 +121,7 @@ def(item)(2)(\
   +redef(PARAGRAPH)(0)(\
     NL()NOTRANS(.RS)+NL()NOTRANS(.PP)+NL()\
     STDPAR()\
-    +redef(ENDITEM)(0)(NOTRANS(.RE)STDPAR())\
+    +redef(ENDITEM)(0)(NOTRANS(.RE)STDPAR()redef(ENDITEM)(0)(CMT()))\
   )\
   ARG2\
   +ENDITEM()\