Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3445)

Issue 3796044: Call ibus_hotkey_profile_remove_hotkey_by_event in bus_ibus_impl_set_trigger (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 5 months ago by Yusuke Sato
Modified:
13 years, 5 months ago
Reviewers:
shawn.p.huang, Peng Huang
CC:
satorux
Base URL:
git@github.com:ibus/ibus.git@master
Visibility:
Public.

Description

Call ibus_hotkey_profile_remove_hotkey_by_event in bus_ibus_impl_set_trigger before registering the fallback hotkey, just in case. Probably it's better to call the remove function in bus_ibus_impl_set_trigger just as bus_ibus_impl_set_hotkey already does, so that a wierd error message like 'IBUS-CRITICAL **: ...: file ibushotkey.c: line 381 (ibus_hotkey_profile_add_hotkey): should not be reached' will not be shown. The fallback hotkey could be registered twice, via bus_ibus_impl_init and _dbus_name_owner_changed_cb, if the --timeout parameter is too small to talk to the configuration daemon, for example. BUG=none TEST=ran ibus-daemon with --timeout=1 and verified that the IBUS-CRITICAL message above was not shown.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M bus/ibusimpl.c View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2
Yusuke Sato
13 years, 5 months ago (2011-01-05 23:28:07 UTC) #1
Peng
13 years, 5 months ago (2011-01-05 23:37:12 UTC) #2
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b