As BusFactoryProxy and IBusInputContext are no longer derived from IBusObject, g_object_ref_sink is not needed after ...
12 years, 11 months ago
(2011-07-14 02:54:23 UTC)
#1
As BusFactoryProxy and IBusInputContext are no longer derived from IBusObject,
g_object_ref_sink is not needed after creation.
Adding ref_sink to buscomp in bus_registry_load_in_dir is for consistency with
bus_registry_load_cache.
BTW, I created systemtap probes to check IBusProxy refs. The log looks mostly
good so far (after the patch is applied):
http://ueno.fedorapeople.org/ibus-stap/
On 2011/07/14 02:54:23, Daiki Ueno wrote: > As BusFactoryProxy and IBusInputContext are no longer derived ...
12 years, 11 months ago
(2011-07-14 12:34:19 UTC)
#2
On 2011/07/14 02:54:23, Daiki Ueno wrote:
> As BusFactoryProxy and IBusInputContext are no longer derived from IBusObject,
> g_object_ref_sink is not needed after creation.
>
> Adding ref_sink to buscomp in bus_registry_load_in_dir is for consistency with
> bus_registry_load_cache.
>
> BTW, I created systemtap probes to check IBusProxy refs. The log looks mostly
> good so far (after the patch is applied):
>
> http://ueno.fedorapeople.org/ibus-stap/
LGTM. Please use git cl push to submit the CL. Thx
Issue 4700048: Fix GObject ref/unref issues.
(Closed)
Created 12 years, 11 months ago by Daiki Ueno
Modified 12 years, 11 months ago
Reviewers: shawn.p.huang_gmail.com, Peng
Base URL: git@github.com:ibus/ibus.git@master
Comments: 0