Merge lp:~ted/indicator-session/gvariant_fixes into lp:indicator-session/0.1

Proposed by Ted Gould
Status: Merged
Merged at revision: 225
Proposed branch: lp:~ted/indicator-session/gvariant_fixes
Merge into: lp:indicator-session/0.1
Diff against target: 31 lines (+2/-2)
2 files modified
src/apt-transaction.c (+0/-1)
src/apt-watcher.c (+2/-1)
To merge this branch: bzr merge lp:~ted/indicator-session/gvariant_fixes
Reviewer Review Type Date Requested Status
Indicator Applet Developers Pending
Review via email: mp+78758@code.launchpad.net

Description of the change

Fixing up some of our g_variant ref counting to hopefully fix a couple of bugs.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/apt-transaction.c'
--- src/apt-transaction.c 2011-09-07 11:04:16 +0000
+++ src/apt-transaction.c 2011-10-09 14:45:27 +0000
@@ -232,7 +232,6 @@
232 0,232 0,
233 current_state);233 current_state);
234 }234 }
235 g_variant_unref (parameters);
236}235}
237236
238static void237static void
239238
=== modified file 'src/apt-watcher.c'
--- src/apt-watcher.c 2011-09-19 20:47:59 +0000
+++ src/apt-watcher.c 2011-10-09 14:45:27 +0000
@@ -425,7 +425,7 @@
425 g_return_if_fail (APT_IS_WATCHER (user_data));425 g_return_if_fail (APT_IS_WATCHER (user_data));
426 AptWatcher* self = APT_WATCHER (user_data);426 AptWatcher* self = APT_WATCHER (user_data);
427427
428 g_variant_ref (parameters);428 g_variant_ref_sink (parameters);
429 GVariant *value = g_variant_get_child_value (parameters, 0);429 GVariant *value = g_variant_get_child_value (parameters, 0);
430430
431 if (g_strcmp0(signal_name, "ActiveTransactionsChanged") == 0){431 if (g_strcmp0(signal_name, "ActiveTransactionsChanged") == 0){
@@ -477,6 +477,7 @@
477 } 477 }
478 } 478 }
479479
480 g_variant_unref (value);
480 g_variant_unref (parameters);481 g_variant_unref (parameters);
481}482}
482483

Subscribers

People subscribed via source and target branches