http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h File bus/factoryproxy.h (right): http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h#newcode80 bus/factoryproxy.h:80: * Get the result of bus_factory_proxy_create_engine call. This call ...
13 years, 7 months ago
(2010-11-17 09:00:33 UTC)
#1
http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h File bus/factoryproxy.h (right): http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h#newcode80 bus/factoryproxy.h:80: * Get the result of bus_factory_proxy_create_engine call. This call ...
13 years, 7 months ago
(2010-11-17 09:31:37 UTC)
#2
Oh I see, thanks. Rewrote the comment. PTAL. On 2010/11/17 09:31:37, Shawn.P.Huang wrote: > http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h ...
13 years, 7 months ago
(2010-11-17 09:38:11 UTC)
#3
Oh I see, thanks. Rewrote the comment.
PTAL.
On 2010/11/17 09:31:37, Shawn.P.Huang wrote:
> http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h
> File bus/factoryproxy.h (right):
>
> http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h#newcode80
> bus/factoryproxy.h:80: * Get the result of bus_factory_proxy_create_engine
call.
> This call might block up to timeout milliseconds.
> On 2010/11/17 09:00:34, Yusuke Sato wrote:
> > > This call might block up to timeout milliseconds.
> >
> > could you check if my understanding is correct?
> No. Caller have to call it in the ready callback function. I will not block
the
> caller.
On 2010/11/17 09:38:11, Yusuke Sato wrote: > Oh I see, thanks. Rewrote the comment. > ...
13 years, 7 months ago
(2010-11-17 09:45:47 UTC)
#4
On 2010/11/17 09:38:11, Yusuke Sato wrote:
> Oh I see, thanks. Rewrote the comment.
> PTAL.
>
> On 2010/11/17 09:31:37, Shawn.P.Huang wrote:
> > http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h
> > File bus/factoryproxy.h (right):
> >
> > http://codereview.appspot.com/3163041/diff/1/bus/factoryproxy.h#newcode80
> > bus/factoryproxy.h:80: * Get the result of bus_factory_proxy_create_engine
> call.
> > This call might block up to timeout milliseconds.
> > On 2010/11/17 09:00:34, Yusuke Sato wrote:
> > > > This call might block up to timeout milliseconds.
> > >
> > > could you check if my understanding is correct?
> > No. Caller have to call it in the ready callback function. I will not block
> the
> > caller.
LGTM. THANKS
Issue 3163041: Add comments to bus/factoryproxy.[ch].
(Closed)
Created 13 years, 7 months ago by Yusuke Sato
Modified 13 years, 7 months ago
Reviewers: Peng Huang, shawn.p.huang_gmail.com, Peng
Base URL: git@github.com:ibus/ibus.git@master
Comments: 2