Merge lp:~mterry/unity/fix-crash-727788 into lp:unity

Proposed by Michael Terry
Status: Rejected
Rejected by: Didier Roche-Tolomelli
Proposed branch: lp:~mterry/unity/fix-crash-727788
Merge into: lp:unity
Diff against target: 30 lines (+3/-3)
1 file modified
services/panel-indicator-entry-accessible.c (+3/-3)
To merge this branch: bzr merge lp:~mterry/unity/fix-crash-727788
Reviewer Review Type Date Requested Status
Rodrigo Moya (community) Disapprove
Review via email: mp+51897@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Rodrigo Moya (rodrigo-moya) wrote :

This fix is wrong, the problem is the call to atk_object_notify_state_change, which shouldn't be there. It's already fixed in this other branch: https://code.launchpad.net/~rodrigo-moya/unity/sync-geometries/+merge/51929 which removes that call

review: Disapprove
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Syncing the state to rejected to clean the list

Unmerged revisions

918. By Michael Terry

make PanelIndicatorEntryAccessible's finalize into dispose to fix crashes

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'services/panel-indicator-entry-accessible.c'
2--- services/panel-indicator-entry-accessible.c 2011-02-28 15:51:27 +0000
3+++ services/panel-indicator-entry-accessible.c 2011-03-02 13:58:19 +0000
4@@ -66,7 +66,7 @@
5 }
6
7 static void
8-panel_indicator_entry_accessible_finalize (GObject *object)
9+panel_indicator_entry_accessible_dispose (GObject *object)
10 {
11 PanelIndicatorEntryAccessible *piea;
12
13@@ -81,7 +81,7 @@
14 g_signal_handlers_disconnect_by_func (piea->priv->service, on_entry_activated_cb, piea);
15 }
16
17- G_OBJECT_CLASS (panel_indicator_entry_accessible_parent_class)->finalize (object);
18+ G_OBJECT_CLASS (panel_indicator_entry_accessible_parent_class)->dispose (object);
19 }
20
21 static void
22@@ -92,7 +92,7 @@
23
24 /* GObject */
25 object_class = G_OBJECT_CLASS (klass);
26- object_class->finalize = panel_indicator_entry_accessible_finalize;
27+ object_class->dispose = panel_indicator_entry_accessible_dispose;
28
29 /* AtkObject */
30 atk_class = ATK_OBJECT_CLASS (klass);