LGTM with one nit below: http://codereview.appspot.com/4446051/diff/1/bus/factoryproxy.c File bus/factoryproxy.c (right): http://codereview.appspot.com/4446051/diff/1/bus/factoryproxy.c#newcode66 bus/factoryproxy.c:66: factory = (BusFactoryProxy *) g_initable_new ( I'd like to know when factory can be NULL. Can you add some comments?
http://codereview.appspot.com/4446051/diff/1/bus/factoryproxy.c File bus/factoryproxy.c (right): http://codereview.appspot.com/4446051/diff/1/bus/factoryproxy.c#newcode66 bus/factoryproxy.c:66: factory = (BusFactoryProxy *) g_initable_new ( On 2011/04/18 14:51:29, Yusuke Sato wrote: > I'd like to know when factory can be NULL. Can you add some comments? It is unlikely. But in some case, the connection could be closed during creating the proxy object. And then g_initable_new will return NULL. https://github.com/ibus/ibus/blob/master/src/ibusproxy.c#L162
http://codereview.appspot.com/4446051/diff/1/bus/factoryproxy.c File bus/factoryproxy.c (right): http://codereview.appspot.com/4446051/diff/1/bus/factoryproxy.c#newcode66 bus/factoryproxy.c:66: factory = (BusFactoryProxy *) g_initable_new ( Thanks for the explanation. Could you copy&paste your comment to the code? On 2011/04/18 15:17:50, Peng wrote: > On 2011/04/18 14:51:29, Yusuke Sato wrote: > > I'd like to know when factory can be NULL. Can you add some comments? > > It is unlikely. But in some case, the connection could be closed during creating > the proxy object. And then g_initable_new will return NULL. > > https://github.com/ibus/ibus/blob/master/src/ibusproxy.c#L162