Re: thrift/concurrency/Mutex.h bug

2012-11-07 Thread Bud Bundy
Thanks, I'd hate to patch this myself every release. https://issues.apache.org/jira/browse/THRIFT-1748 Oh and sorry, I couldn't find "attach" in JIRA, maybe I don't have permissions, so I had to post the code. On Wed, Nov 7, 2012 at 5:23 AM, Henrique Mendonça wrote: > Hi Bud, > > I wonder why we

Re: thrift/concurrency/Mutex.h bug

2012-11-07 Thread Ben Craig
onça To: u...@thrift.apache.org Cc: dev@thrift.apache.org Date: 11/07/2012 04:24 AM Subject: Re: thrift/concurrency/Mutex.h bug Sent by:hmendo...@gmail.com Hi Bud, I wonder why we are not using boost mutex and scoped mutex instead but those two macros are probably not necessary

Re: thrift/concurrency/Mutex.h bug

2012-11-07 Thread Henrique Mendonça
Hi Bud, I wonder why we are not using boost mutex and scoped mutex instead but those two macros are probably not necessary anyways. Would you create a patch and copy/adapt those comments to the thrift Mutex classes themselves? Thank you! http://thrift.apache.org/docs/HowToContribute/ Henrique On