Re: New yodl warnings with CVS

[Peter Stephenson <pws@csr.com>]

On Fri, 14 Dec 2007 00:43:16 +0200
Ismail Dönmez <ismail@pardus.org.tr> wrote:
> Hi all,
> 
> I am getting these warning from yodl lately,
> 
> expn.yo:1065: No macro: time(...)
> calsys.yo:494: No macro: timeNL(...)
> calsys.yo:494: No macro: warntimeNL(...)
> calsys.yo:494: No macro: warnstrNL(...)
> calsys.yo:494: No macro: rpttimeNL(...)
> calsys.yo:494: No macro: rptstrNL(...)
> 
> Any ideas?

I've been seeing these intermittently, but every time I look more closely
they go away, so I haven't been very sedulous in their pursuit.

I *think* the answer is the following.  The macro has always been slackly
defined (c.f. the definition of sitem in zman.yo) but it's only recently
been used in a way that makes that obvious.

Index: Doc/ztexi.yo
===================================================================
RCS file: /cvsroot/zsh/zsh/Doc/ztexi.yo,v
retrieving revision 1.5
diff -u -r1.5 ztexi.yo
--- Doc/ztexi.yo	20 Mar 2006 11:06:24 -0000	1.5
+++ Doc/ztexi.yo	14 Dec 2007 11:22:06 -0000
@@ -183,7 +183,7 @@
   enditem()\
 )
 def(sitem)(2)(\
-  item(ARG1)(NL()ARG2)\
+  item(ARG1)(+NL()ARG2)\
 )
 def(sxitem)(1)(\
   xitem(ARG1)\



-- 
Peter Stephenson <pws@csr.com>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070