Mailing-List: contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-apps-owner AT cygwin DOT com
List-Subscribe: <mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-apps/>
List-Post: <mailto:cygwin-apps AT cygwin DOT com>
List-Help: <mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Mail-Followup-To: cygwin-apps AT cygwin DOT com
Delivered-To: mailing list cygwin-apps AT cygwin DOT com
Subject: RE: Setup 2.218.2.6 [Was: Re: Setup 2.218.2.4] (new version uploaded)
Date: Wed, 15 May 2002 17:07:53 +1000
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Message-ID: <FC169E059D1A0442A04C40F86D9BA7600C60BA@itdomain003.itdomain.net.au>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
X-MimeOLE: Produced By Microsoft Exchange V6.0.5762.3
content-class: urn:content-classes:message
From: "Robert Collins" <robert DOT collins AT itdomain DOT com DOT au>
To: <cygwin-apps AT cygwin DOT com>
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by delorie.com id g4F781i12447



> -----Original Message-----
> From: Christopher Faylor [mailto:cgf AT redhat DOT com] 
> Sent: Wednesday, May 15, 2002 8:55 AM


> The attached patch seems to fix this and some other problems.
> 
> I've uploaded a new version to sourceware.
> 
> At least some of the below should be useful on the trunk, too.

Thanks heaps Chris. I appreciate this. (BTW: how did you reproduce the
fault?)

I'll probably rearrange what you've put into HEAD a little, as static
variables are getting eliminated (the Builder is the appropriate place
to store such information).

I notice that you've increased the stack size for the parser - this
shouldn't have been needed with the right hand recursion removed. Did
you observe continued overflows or was this preventative?

Cheers,
Rob