This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: Patch for Review: PR/9254 and other Win32 Threading Fixes


It looks fine to me, too.

If someone wants to further tune this, it looks to me like the EnterCriticalSection and LeaveCriticalSection calls could be avoided for nested lock acquisitions, as they are in the Posix case.  It seems that the fact that EnterCriticalSection is already reentrant doesn't really buy you anything, and you'd be better off pretending it weren't.  Probably Adam's suggestion to migrate more of this into the platform-independent code is right.

Hans

> -----Original Message-----
> From: Adam Megacz [mailto:gcj@lists.megacz.com]
> Sent: Tuesday, January 28, 2003 12:11 PM
> To: Ranjit Mathew
> Cc: java-patches@gcc.gnu.org
> Subject: Re: Patch for Review: PR/9254 and other Win32 Threading Fixes
> 
> 
> 
> Looks good to me.
> 
>   - a
> 
> 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]