[PATCH] stdio initialization

Thomas Pfaff tpfaff@gmx.net
Mon Nov 10 15:18:00 GMT 2003


Christopher Faylor wrote:
> On Mon, Nov 10, 2003 at 03:03:06PM +0100, Thomas Pfaff wrote:
> 
>>Christopher Faylor wrote:
>>
>>>On Mon, Nov 10, 2003 at 12:23:35PM +0100, Thomas Pfaff wrote:
>>>
>>>
>>>>Attached patch fixes the memory leak reported by Arash Partow by
>>>>initializing stdio during startup and setting __sdidinit from thread
>>>>local clib appropriately.
>>>>
>>>>Thomas
>>>>
>>>>2003-11-10  Thomas Pfaff  <tpfaff@gmx.net>
>>>>
>>>>	* dcrt0.cc: Add prototype for __sinit.
>>>>	(dll_crt0_1): Initialize stdio.
>>>
>>>
>>>The above two things are already done in dcrt0.cc.  Why are you adding
>>>additional prototypes and going to additional work?
>>>
>>
>>Ouch. I should have stayed in bed today.
> 
> But the rest of your patch looks ok, I think, doesn't it?

After i removed the tomatoes from my eyes and looked again:
Yes, it does.
I will commit to CVS in about 4 hours when i am at home.

Thomas



More information about the Cygwin-patches mailing list