http://codereview.appspot.com/3156041/diff/1/bus/dbusimpl.c File bus/dbusimpl.c (right): http://codereview.appspot.com/3156041/diff/1/bus/dbusimpl.c#newcode1080 bus/dbusimpl.c:1080: if (G_UNLIKELY (IBUS_OBJECT_DESTROYED (dbus))) question: is this check thread-safe? i'm wondering if it could happen that the dbus object is deleted in the main thread right after the check. http://codereview.appspot.com/3156041/diff/1/bus/dbusimpl.c#newcode1182 bus/dbusimpl.c:1182: if (G_UNLIKELY (IBUS_OBJECT_DESTROYED (dbus))) { question: (the same as above.)
Talked with Peng offline, and added FIXME to the function. Peng, pls check the patch set #2. does this look good to you? On 2010/11/17 04:41:16, Yusuke Sato wrote: > http://codereview.appspot.com/3156041/diff/1/bus/dbusimpl.c > File bus/dbusimpl.c (right): > > http://codereview.appspot.com/3156041/diff/1/bus/dbusimpl.c#newcode1080 > bus/dbusimpl.c:1080: if (G_UNLIKELY (IBUS_OBJECT_DESTROYED (dbus))) > question: is this check thread-safe? i'm wondering if it could happen that the > dbus object is deleted in the main thread right after the check. > > http://codereview.appspot.com/3156041/diff/1/bus/dbusimpl.c#newcode1182 > bus/dbusimpl.c:1182: if (G_UNLIKELY (IBUS_OBJECT_DESTROYED (dbus))) { > question: (the same as above.)
LGTM. Thanks