https://codereview.appspot.com/321610043/diff/1/ui/gtk3/emojier.vala File ui/gtk3/emojier.vala (right): https://codereview.appspot.com/321610043/diff/1/ui/gtk3/emojier.vala#newcode1566 ui/gtk3/emojier.vala:1566: if (!(this is IBusEmojier)) why we need it? https://codereview.appspot.com/321610043/diff/1/ui/gtk3/switcher.vala ...
6 years, 7 months ago
(2017-09-08 15:09:56 UTC)
#2
https://codereview.appspot.com/321610043/diff/1/ui/gtk3/emojier.vala File ui/gtk3/emojier.vala (right): https://codereview.appspot.com/321610043/diff/1/ui/gtk3/emojier.vala#newcode1566 ui/gtk3/emojier.vala:1566: if (!(this is IBusEmojier)) On 2017/09/11 03:17:28, fujiwara wrote: ...
6 years, 7 months ago
(2017-09-11 13:50:30 UTC)
#4
https://codereview.appspot.com/321610043/diff/1/ui/gtk3/emojier.vala
File ui/gtk3/emojier.vala (right):
https://codereview.appspot.com/321610043/diff/1/ui/gtk3/emojier.vala#newcode1566
ui/gtk3/emojier.vala:1566: if (!(this is IBusEmojier))
On 2017/09/11 03:17:28, fujiwara wrote:
> On 2017/09/08 15:09:56, Peng wrote:
> > why we need it?
>
> Because the focus in event can be called before either emojier instructor is
> completed or destructor is completed.
> You could see the backtrace of the original bug.
I feel it is not right to call an object method before construction or after
destruction. It must be a bug in other place. It is not good to test it here.
Issue 321610043: ui/gtk3: Fix SEGV in IBusEmojier.get_selected_string
Created 6 years, 7 months ago by fujiwara
Modified 6 years, 7 months ago
Reviewers: shawn.p.huang_gmail.com
Base URL: git@github.com:ibus/ibus.git@master
Comments: 4