Unless s has been toUpperCase() (which I did not explictly check), then this
is not spec compliant.  It needs to check for "true" and "false" too.

Do you know what is used to generate parser.java from jms.y?

--jason


On Wed, 22 Aug 2001, Juha-P Lindfors wrote:

> It's here:
> 
>http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/jboss/spyderMQ/src/java/org/spydermq/selectors/
>
> this is the boolean equal check:
>
>                 //CST group
>                 if (s.equals("TRUE")) {
>                         yylval = new parserval((Object)Boolean.TRUE);
>                         return CST;
>                 }
>                 if (s.equals("FALSE")) {
>                         yylval = new parserval((Object)Boolean.FALSE);
>                         return CST;
>                 }
>
> -- Juha
>
>
>
> On Wed, 22 Aug 2001, Hiram Chirino wrote:
>
> >
> > Can you send it to me???  I'll check it in.
> >
> > Regards,
> > Hiram
> >
> > >From: Juha-P Lindfors <[EMAIL PROTECTED]>
> > >Reply-To: [EMAIL PROTECTED]
> > >To: <[EMAIL PROTECTED]>
> > >Subject: Re: [JBoss-dev] jbossmq selector parser grammer source
> > >Date: Wed, 22 Aug 2001 13:01:02 +0300 (EET DST)
> > >
> > >
> > > > > I have just fixed (and verified) both of these problems.  Should I
> > >commit
> > > > > them?  Silly me, of course I should commit them... but where is jms.y?
> > >
> > >I have the jms.y as part of the old spyderMQ module in
> > >src/java/org/spydermq/selectors.
> > >
> > >No idea where it is in the newer modules, or if it was ever transferred.
> > >But check the SpyderMQ in CVS.
> > >
> > >-- Juha
> > >
> > >
> > >_______________________________________________
> > >Jboss-development mailing list
> > >[EMAIL PROTECTED]
> > >http://lists.sourceforge.net/lists/listinfo/jboss-development
> >
> >
> > _________________________________________________________________
> > Get your FREE download of MSN Explorer at http://explorer.msn.com/intl.asp
> >
> >
> > _______________________________________________
> > Jboss-development mailing list
> > [EMAIL PROTECTED]
> > http://lists.sourceforge.net/lists/listinfo/jboss-development
> >
>
>
> _______________________________________________
> Jboss-development mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-development
>


_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to