> -----Original Message-----
> From: Christopher Faylor [mailto:[EMAIL PROTECTED]]
> Sent: Saturday, April 20, 2002 12:42 AM
> To: [EMAIL PROTECTED]
> Subject: Re: [PATCH] dtors run twice on dll detach (update)
>
>
> On Fri, Apr 19, 2002 at 10:46:28PM +1000, Robert Collins wrote:
> >
> >
> >> -----Original Message-----
> >> From: Thomas Pfaff [mailto:[EMAIL PROTECTED]]
> >> Sent: Wednesday, April 17, 2002 5:08 PM
> >
> >> > Since i can not judge which function is obsolete (i guess
> >> > dll_global_dtors
> >> > is) i have attached a small patch that will make sure that
> >> the dtors run
> >> > only once.
> >
> >I'm not sure that either function is obsolete - I'll let
> Chris/Corinna
> >comment on that.. Your patch looked good, and corrected a
> test case I
> >happened to have hanging around, so I've checked this in as an
> >appropriate solution. Thanks for the patch.
>
> If one of the functions is obsolete, it should be deleted.
> That means that the patch does *not* look good. It needs to
> be reviewed.
>
> That plus the fact that you don't have global checkin
> privileges for cygwin == cgf reverts the patch.
Ookay. I don't think that either function is obsolete... and neither you
nor Corinna had commented.
Anyay, I'v reviewed, given feedback, it's in your court.
Rob
