Add asynchronous APIs to ibusbus.h

Add asynchronous APIs to ibusbus.h

Modified src/tests/ibus-bus.c so that it could be compiled with ibus-1.3.99 as well.

BUG=chromium-os:12581
TEST=added async api tests to the src/tests/ibus-bus.c and ran it locally.

[Peng Huang, 2011-03-01 16:52:30]

http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c
File src/ibusbus.c (right):

http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c#newcode423
src/ibusbus.c:423: return g_strdup (path);
I think maybe we don't need strdup the path, and just return const gchar*;

[Yusuke Sato, 2011-03-01 17:01:06]

http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c
File src/ibusbus.c (right):

http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c#newcode423
src/ibusbus.c:423: return g_strdup (path);
On 2011/03/01 16:52:30, Peng Huang wrote:
> I think maybe we don't need strdup the path, and just return const gchar*;

Done.

[Yusuke Sato, 2011-03-01 20:08:03]

Almost done (patch set #4). I've added async versions for all APIs.

I'll write header comments and tests when I get back in Tokyo, and then will
start (official) code review.

On 2011/03/01 17:01:06, Yusuke Sato wrote:
> http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c
> File src/ibusbus.c (right):
> 
> http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c#newcode423
> src/ibusbus.c:423: return g_strdup (path);
> On 2011/03/01 16:52:30, Peng Huang wrote:
> > I think maybe we don't need strdup the path, and just return const gchar*;
> 
> Done.

[Yusuke Sato, 2011-03-06 14:03:38]

Peng, all done. Please review.

On 2011/03/01 20:08:03, Yusuke Sato wrote:
> Almost done (patch set #4). I've added async versions for all APIs.
> 
> I'll write header comments and tests when I get back in Tokyo, and then will
> start (official) code review.
> 
> On 2011/03/01 17:01:06, Yusuke Sato wrote:
> > http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c
> > File src/ibusbus.c (right):
> > 
> > http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c#newcode423
> > src/ibusbus.c:423: return g_strdup (path);
> > On 2011/03/01 16:52:30, Peng Huang wrote:
> > > I think maybe we don't need strdup the path, and just return const gchar*;
> > 
> > Done.

[Peng Huang, 2011-03-07 15:42:17]

LGTM.

On 2011/03/06 14:03:38, Yusuke Sato wrote:
> Peng, all done. Please review.
> 
> On 2011/03/01 20:08:03, Yusuke Sato wrote:
> > Almost done (patch set #4). I've added async versions for all APIs.
> > 
> > I'll write header comments and tests when I get back in Tokyo, and then will
> > start (official) code review.
> > 
> > On 2011/03/01 17:01:06, Yusuke Sato wrote:
> > > http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c
> > > File src/ibusbus.c (right):
> > > 
> > > http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c#newcode423
> > > src/ibusbus.c:423: return g_strdup (path);
> > > On 2011/03/01 16:52:30, Peng Huang wrote:
> > > > I think maybe we don't need strdup the path, and just return const
gchar*;
> > > 
> > > Done.

[Peng Huang, 2011-03-07 17:59:28]

Found a problem. make dpkg failed. Because of lack new symbols in
debian/libibus-xx.symbols

On 2011/03/07 15:42:17, Peng Huang wrote:
> LGTM.
> 
> On 2011/03/06 14:03:38, Yusuke Sato wrote:
> > Peng, all done. Please review.
> > 
> > On 2011/03/01 20:08:03, Yusuke Sato wrote:
> > > Almost done (patch set #4). I've added async versions for all APIs.
> > > 
> > > I'll write header comments and tests when I get back in Tokyo, and then
will
> > > start (official) code review.
> > > 
> > > On 2011/03/01 17:01:06, Yusuke Sato wrote:
> > > > http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c
> > > > File src/ibusbus.c (right):
> > > > 
> > > > http://codereview.appspot.com/4179061/diff/2001/src/ibusbus.c#newcode423
> > > > src/ibusbus.c:423: return g_strdup (path);
> > > > On 2011/03/01 16:52:30, Peng Huang wrote:
> > > > > I think maybe we don't need strdup the path, and just return const
> gchar*;
> > > > 
> > > > Done.