thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Henrique Mendonça <>
Subject Re: thrift/concurrency/Mutex.h bug
Date Wed, 07 Nov 2012 10:23:21 GMT
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!


On 7 November 2012 00:19, Bud Bundy <> wrote:

> Hi,
> I'm using version 0.9.0.  The following code in thrift/concurrency/Mutex.h
> is causing me great headaches:
> // A little hack to prevent someone from trying to do "Guard(m);"
> // Such a use is invalid because the temporary Guard object is
> // destroyed at the end of the line, releasing the lock.
> // Sorry for polluting the global namespace, but I think it's worth it.
> #define Guard(m) incorrect_use_of_Guard(m)
> #define RWGuard(m) incorrect_use_of_RWGuard(m)
> The global macro is interfering with another library I'm using that also
> defines a Guard class within their own namespace.  So, no, I don't think
> it's worth it.  I think it's a bug.  I can patch the file by removing those
> two lines, but any way this can be fixed in the trunk?
> Thanks.

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message