[Fwd: Re: [Opencxx-users] parser/PtreeBrace.cc]

[Grzegorz J. <ja...@ac...>]

-------- Original Message --------
Subject: Re: [Opencxx-users] parser/PtreeBrace.cc
Date: Wed, 11 Aug 2004 23:10:03 +0800
From: Grzegorz Jakacki <ja...@ac...>
To: Gilles J. Seguin <se...@in...>
References: <109...@se...>



Gilles J. Seguin wrote:
> Grzegorz,
> 
> Can you verify this.
> function is call second() but return the third element 
> of the list.
> get_body() would have being a better name.

Hi,

I am not sure if I understand. PtreeUtil::Second(), as it stands both in
HEAD and rel_2_8, returns second element of the list (or NULL if it does
not exist). After applying the patch from your e-mail (shown below), it
will indeed return third element. This is a patch against what?

BR
Grzegorz

> 
> Since the result from qmtest are hardcoded,
> $ qmtest -o myresults.qmr run  # little 'o'
> on future test do
> $ qmtest -O myresults.qmr run   # uppercase 'O'

Where is the version with qmtest?

BR
Grzegorz

> 
> diff -u -r1.2.2.1 PtreeBrace.cc
> --- PtreeBrace.cc       8 Jul 2004 11:57:13 -0000       1.2.2.1
> +++ PtreeBrace.cc       10 Aug 2004 13:13:03 -0000
> @@ -59,10 +59,10 @@
>      Ptree* Second(Ptree* p)
>      {
>          if (p) {
> -            p=p->Cdr();
> -            if (p) {
> +            //p=p->Cdr();
> +            //if (p) {
>                  return p->Cdr();
> -            }
> +            //}
>          }
>          return p;
>      }
> 
> 
> 
> 
> 
> -------------------------------------------------------
> SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
> 100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
> Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
> http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
> _______________________________________________
> Opencxx-users mailing list
> Ope...@li...
> https://lists.sourceforge.net/lists/listinfo/opencxx-users