diff options
author | Colin Walters <walters@verbum.org> | 2011-11-22 10:20:10 -0500 |
---|---|---|
committer | Colin Walters <walters@verbum.org> | 2011-11-22 10:20:10 -0500 |
commit | 3c025e526f5a501414235c4dd5aa29162c1b6251 (patch) | |
tree | c4c8429e29651f9a4778b2bedefb297112f14a89 /src/ostree/ot-builtin-log.c | |
parent | 43ae898f6529f29c714b8093c3fd3e0b15a609f3 (diff) | |
download | ostree-3c025e526f5a501414235c4dd5aa29162c1b6251.tar.gz |
core: Ad ot_clear_gvariant(), use it
This saves us a conditional.
Diffstat (limited to 'src/ostree/ot-builtin-log.c')
-rw-r--r-- | src/ostree/ot-builtin-log.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/ostree/ot-builtin-log.c b/src/ostree/ot-builtin-log.c index fe3723e7..41a1ae56 100644 --- a/src/ostree/ot-builtin-log.c +++ b/src/ostree/ot-builtin-log.c @@ -85,8 +85,7 @@ ostree_builtin_log (int argc, char **argv, const char *repo_path, GError **error GVariant *commit_metadata = NULL; char *formatted_metadata = NULL; - if (commit) - g_variant_unref (commit); + ot_clear_gvariant (&commit); if (!ostree_repo_load_variant (repo, resolved_rev, &type, &commit, error)) goto out; @@ -101,8 +100,7 @@ ostree_builtin_log (int argc, char **argv, const char *repo_path, GError **error g_date_time_unref (time_obj); time_obj = NULL; - formatted_metadata = g_variant_print (commit_metadata, TRUE); - g_variant_unref (commit_metadata); + ot_clear_gvariant (&commit_metadata); formatted = g_strdup_printf ("commit %s\nSubject: %s\nDate: %s\nMetadata: %s\n\n", resolved_rev, subject, formatted_date, formatted_metadata); g_free (formatted_metadata); @@ -147,8 +145,7 @@ ostree_builtin_log (int argc, char **argv, const char *repo_path, GError **error g_free (resolved_rev); if (context) g_option_context_free (context); - if (commit) - g_variant_unref (commit); + ot_clear_gvariant (&commit); g_clear_object (&repo); return ret; } |