On 2017/04/13 07:10:02, fujiwara wrote:
> It's an interesting patch.
> Now the dbus error is fixed and ibus-ui-emojier no longer exit by --help
option.
>
>
> How about adding "ibus emoji exit" option to exit the backend while you
already
> add a timeout of it?
>
For unique applications, the unique app needs to send back an dbus reply to
the second instance.
Implementing the "ibus emoji exit" command, will bring back the dbus error.
> https://codereview.appspot.com/320480043/diff/1/ui/gtk3/emojierapp.vala
> File ui/gtk3/emojierapp.vala (right):
>
>
https://codereview.appspot.com/320480043/diff/1/ui/gtk3/emojierapp.vala#newco...
> ui/gtk3/emojierapp.vala:28: private IBus.Emojier emojier = new IBus.Emojier();
> Today I noticed this.
> Could you change s/IBus.Emojier emojier/IBus.Emojier m_emojier/ ?
Fixing it.
On 2017/04/13 07:35:06, Peng Wu wrote:
> On 2017/04/13 07:10:02, fujiwara wrote:
> > It's an interesting patch.
> > Now the dbus error is fixed and ibus-ui-emojier no longer exit by --help
> option.
> >
> >
> > How about adding "ibus emoji exit" option to exit the backend while you
> already
> > add a timeout of it?
> >
>
> For unique applications, the unique app needs to send back an dbus reply to
> the second instance.
>
> Implementing the "ibus emoji exit" command, will bring back the dbus error.
OK, there may be a better fix. Let's leave this CL until we get time.
>
> > https://codereview.appspot.com/320480043/diff/1/ui/gtk3/emojierapp.vala
> > File ui/gtk3/emojierapp.vala (right):
> >
> >
>
https://codereview.appspot.com/320480043/diff/1/ui/gtk3/emojierapp.vala#newco...
> > ui/gtk3/emojierapp.vala:28: private IBus.Emojier emojier = new
IBus.Emojier();
> > Today I noticed this.
> > Could you change s/IBus.Emojier emojier/IBus.Emojier m_emojier/ ?
>
> Fixing it.
Issue 320480043: Fixes ibus-ui-emojier dbus error when print help
Created 7 years ago by Peng Wu
Modified 7 years ago
Reviewers: shawn.p.huang_gmail.com, fujiwara
Base URL:
Comments: 1