Merge lp:~jpetersen/netbook-remix-launcher/fix-bug-465548 into lp:netbook-remix-launcher

Proposed by Jan Arne Petersen
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jpetersen/netbook-remix-launcher/fix-bug-465548
Merge into: lp:netbook-remix-launcher
Diff against target: 20 lines (+6/-0)
1 file modified
netbook-launcher/nl-content-view.c (+6/-0)
To merge this branch: bzr merge lp:~jpetersen/netbook-remix-launcher/fix-bug-465548
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+21555@code.launchpad.net

Description of the change

Reset priv->old_view to NULL at the end of the fade-out animation in NlContentView.

Fixes: lp#465548 - netbook-launcher crashed with SIGSEGV in
g_cclosure_marshal_VOID__VOID() - i was just clicking around

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Thanks for your merge proposal. Awesome! merging in trunk now.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'netbook-launcher/nl-content-view.c'
2--- netbook-launcher/nl-content-view.c 2009-07-23 14:58:31 +0000
3+++ netbook-launcher/nl-content-view.c 2010-03-17 13:24:19 +0000
4@@ -201,10 +201,16 @@
5 on_old_hidden (ClutterAnimation *anim,
6 NlContentView *view)
7 {
8+ NlContentViewPrivate *priv;
9+
10 if (NL_IS_CONTENT_VIEW (view))
11 {
12 ClutterActor *child = (ClutterActor*)clutter_animation_get_object (anim);
13
14+ priv = view->priv;
15+
16+ priv->old_child = NULL;
17+
18 if (ctk_bin_get_child (CTK_BIN (view)) != child)
19 clutter_actor_unparent (child);
20 }

Subscribers

People subscribed via source and target branches