New yodl warnings with CVS

New yodl warnings with CVS

[Ismail Dönmez]

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?

Regards,
ismail

-- 
Never learn by your mistakes, if you do you may never dare to try again.

Re: New yodl warnings with CVS

[Peter Stephenson]

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

Re: New yodl warnings with CVS

[Ismail Dönmez]

Friday 14 December 2007 13:24:29 tarihinde Peter Stephenson şunları yazmıştı:
> 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.

Indeed only one warning left:

expn.yo:1065: No macro: time(...)

Thanks!

-- 
Never learn by your mistakes, if you do you may never dare to try again.

Re: New yodl warnings with CVS

[Ismail Dönmez]

Friday 14 December 2007 21:47:19 tarihinde şunları yazmıştınız:
> Friday 14 December 2007 13:24:29 tarihinde Peter Stephenson şunları 
yazmıştı:
> > 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.
>
> Indeed only one warning left:
>
> expn.yo:1065: No macro: time(...)

Your latest commit fixed this, thanks.

Regards,
ismail

-- 
Never learn by your mistakes, if you do you may never dare to try again.