On 2011/09/16 16:52:38, Peng wrote: LGTM. Thanks for tightening up the error handling.
LGTM http://codereview.appspot.com/5050041/diff/1/src/engine.c File src/engine.c (right): http://codereview.appspot.com/5050041/diff/1/src/engine.c#newcode822 src/engine.c:822: IBusText *text = ibus_text_new_from_unichar (buf[i]); when the function returns null?