Move the log handler code from bus/main.cc to src/ibusshare.cc.

Move the log handler code from bus/main.cc to src/ibusshare.cc.

The log handler is useful for IBus client programs. For instance,
the log handler was used inside the chromium browser for debugging
IBus startup issue:

http://code.google.com/p/chromium-os/issues/detail?id=6375#c29

(chrome:11824): IBUS-DEBUG: 00:00:46.929430: The socket file is now created (monitoring done).


BUG=none
TEST=manually ran ibus-daemon with and without -v

[Peng Huang, 2010-09-08 07:56:22]

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.c
File src/ibusshare.c (right):

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.c#newcode2
src/ibusshare.c:2: /* vim:set et sts=4: */
Does this work for vim?

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.h
File src/ibusshare.h (right):

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.h#newcode336
src/ibusshare.h:336: void             ibus_set_log_handler   (gboolean verbose);
If the program does not call this function, what is the default behavior?

[satorux, 2010-09-08 08:11:12]

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.c
File src/ibusshare.c (right):

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.c#newcode2
src/ibusshare.c:2: /* vim:set et sts=4: */
On 2010/09/08 07:56:22, Peng Huang wrote:
> Does this work for vim?

Yes, it works as long as you have "set modeline" in .vimrc, but I didn't mean to
include this in this patch. Removed.

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.h
File src/ibusshare.h (right):

http://codereview.appspot.com/2122047/diff/1/src/ibusshare.h#newcode336
src/ibusshare.h:336: void             ibus_set_log_handler   (gboolean verbose);
On 2010/09/08 07:56:22, Peng Huang wrote:
> If the program does not call this function, what is the default behavior?

There is no problem. Added a comment about it.

[Peng Huang, 2010-09-08 08:17:15]

LGTM

[satorux, 2010-09-08 09:08:51]

As chatted offline, reworked the logic. please take another look

On 2010/09/08 08:17:15, Peng Huang wrote:
> LGTM

[Peng Huang, 2010-09-08 09:42:15]

LGTM