LGTM with one nit. http://codereview.appspot.com/5051043/diff/2001/src/ibustext.c File src/ibustext.c (right): http://codereview.appspot.com/5051043/diff/2001/src/ibustext.c#newcode166 src/ibustext.c:166: g_return_val_if_reached (NULL); g_return_val_fail (buf != NULL, NULL); (and remove the if block) would be better.
http://codereview.appspot.com/5051043/diff/2001/src/ibustext.c File src/ibustext.c (right): http://codereview.appspot.com/5051043/diff/2001/src/ibustext.c#newcode166 src/ibustext.c:166: g_return_val_if_reached (NULL); On 2011/09/18 08:02:44, Yusuke Sato wrote: > g_return_val_fail (buf != NULL, NULL); (and remove the if block) would be > better. Done.