delorie.com/archives/browse.cgi   search  
Mail Archives: cygwin-apps/2002/03/29/04:44:43

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
content-class: urn:content-classes:message
Subject: RE: Re[2]: Patch for Setup.exe problem and for mklink2.cc
MIME-Version: 1.0
Date: Fri, 29 Mar 2002 20:44:23 +1100
Message-ID: <FC169E059D1A0442A04C40F86D9BA76008ABEA@itdomain003.itdomain.net.au>
X-MimeOLE: Produced By Microsoft Exchange V6.0.5762.3
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
From: "Robert Collins" <robert DOT collins AT itdomain DOT com DOT au>
To: "Pavel Tsekov" <ptsekov AT syntrex DOT com>
Cc: <cygwin-apps AT cygwin DOT com>
X-MIME-Autoconverted: from quoted-printable to 8bit by delorie.com id g2T9igD22194


> -----Original Message-----
> From: Pavel Tsekov [mailto:ptsekov AT syntrex DOT com] 
> Sent: Friday, March 29, 2002 8:42 PM
> To: Robert Collins
> Cc: cygwin-apps AT cygwin DOT com
> Subject: Re[2]: Patch for Setup.exe problem and for mklink2.cc
> 
> 
> Hello Robert,
> 
> Friday, March 29, 2002, 10:25:08 AM, you wrote:
> 
> >> > Found the problem causing the segment violation and probably 
> >> > causing Jonas Eriksson's problem. It is a typical case 
> of 'off by 
> >> > 1'. In PickView::set_headers the loop filling the window header 
> >> > does one iteration too much, resulting in a call to DoInsertItem 
> >> > with a NULL string pointer and hence a crash following. While 
> >> > debugging this I could not compile the new mklink2.cc ( the
> >> > c++ version of the original mklink2.c). It seems three & 
> (address 
> >> > c++ of
> >> > c++ operator)
> >> > have disappeared in the transition. Putting them back made the 
> >> > compiler happy. Is this OK Robert ?
> >> 
> >> I'll check the off-by-one fix in tomorrow, as I'm off to bed now.
> 
> RC> Actually, this is not an off-by-one error. It's an invalid data 
> RC> error - the loop is meant to go from 0 to == last_col, where last 
> RC> col is meant to be equal to the greatest column number. The first 
> RC> set of column details had last_col set incorrectly.
> 
> Btw I don't think my question is related to this is issue but 
> still... I notice there is a dummy column in the list view 
> after the "Package" column - is this a feature ? If so what 
> is its purpose ?

There shouldn't be - but HEAD doesn't have the fix for the data yet, you
might try the released setup.exe and see if it looks the same to you.

Rob

- Raw text -


  webmaster     delorie software   privacy  
  Copyright 2019   by DJ Delorie     Updated Jul 2019