This is the mail archive of the
cygwin-apps@cygwin.com
mailing list for the Cygwin project.
RE: Patch for Setup.exe problem and for mklink2.cc
- From: "Robert Collins" <robert dot collins at itdomain dot com dot au>
- To: "Gary R Van Sickle" <tiberius at braemarinc dot com>,<cygwin-apps at cygwin dot com>
- Date: Thu, 28 Mar 2002 07:21:02 +1100
- Subject: RE: Patch for Setup.exe problem and for mklink2.cc
> -----Original Message-----
> From: cygwin-apps-owner@cygwin.com
> [mailto:cygwin-apps-owner@cygwin.com] On Behalf Of Gary R Van Sickle
> Sent: Thursday, March 28, 2002 4:48 AM
> To: cygwin-apps@cygwin.com
> Subject: RE: Patch for Setup.exe problem and for mklink2.cc
>
>
> [snip]
>
> > > 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 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.
> >
>
> [snip]
>
> There have been a few of these off-by-ones in Setup. Is it
> possible your habit of using 1-based arrays is a contributing
> factor Rob? This isn't a convention I see much (modulo
> "Numerical Recipies"); is there a reason you use this convention?
Errm, there aren't any 1 based arrayts at the moment IIRC. Are do you
mean the list class with the index postion as the parameter?
Well that's different, uhmm, yeah, yeah it is.
Rob