diff options
author | Colin Walters <walters@verbum.org> | 2022-06-14 09:50:07 -0400 |
---|---|---|
committer | Colin Walters <walters@verbum.org> | 2022-06-15 09:35:03 -0400 |
commit | 2f1c9a727e598f206eae08412de2a872c0a9dedf (patch) | |
tree | 0c4c50f691beead2eea86f4c4a767a4a67b244a3 /src/ostree/ot-remote-builtin-list.c | |
parent | a0ae2f9156982a94b2ad163c5bf61184c1ffcb5e (diff) | |
download | ostree-2f1c9a727e598f206eae08412de2a872c0a9dedf.tar.gz |
cli/remote-list: Port to C99 style
General background cleanup.
Diffstat (limited to 'src/ostree/ot-remote-builtin-list.c')
-rw-r--r-- | src/ostree/ot-remote-builtin-list.c | 27 |
1 files changed, 10 insertions, 17 deletions
diff --git a/src/ostree/ot-remote-builtin-list.c b/src/ostree/ot-remote-builtin-list.c index 552e5359..b8e875d4 100644 --- a/src/ostree/ot-remote-builtin-list.c +++ b/src/ostree/ot-remote-builtin-list.c @@ -37,45 +37,38 @@ static GOptionEntry option_entries[] = { gboolean ot_remote_builtin_list (int argc, char **argv, OstreeCommandInvocation *invocation, GCancellable *cancellable, GError **error) { - g_autoptr(GOptionContext) context = NULL; - g_autoptr(OstreeRepo) repo = NULL; - g_auto(GStrv) remotes = NULL; - guint ii, n_remotes = 0; - gboolean ret = FALSE; - - context = g_option_context_new (""); + g_autoptr(GOptionContext) context = g_option_context_new (""); + g_autoptr(OstreeRepo) repo = NULL; if (!ostree_option_context_parse (context, option_entries, &argc, &argv, invocation, &repo, cancellable, error)) - goto out; + return FALSE; - remotes = ostree_repo_remote_list (repo, &n_remotes); + guint n_remotes = 0; + g_auto(GStrv) remotes = ostree_repo_remote_list (repo, &n_remotes); if (opt_show_urls) { int max_length = 0; - for (ii = 0; ii < n_remotes; ii++) + for (guint ii = 0; ii < n_remotes; ii++) max_length = MAX (max_length, strlen (remotes[ii])); - for (ii = 0; ii < n_remotes; ii++) + for (guint ii = 0; ii < n_remotes; ii++) { g_autofree char *remote_url = NULL; if (!ostree_repo_remote_get_url (repo, remotes[ii], &remote_url, error)) - goto out; + return FALSE; g_print ("%-*s %s\n", max_length, remotes[ii], remote_url); } } else { - for (ii = 0; ii < n_remotes; ii++) + for (guint ii = 0; ii < n_remotes; ii++) g_print ("%s\n", remotes[ii]); } - ret = TRUE; - - out: - return ret; + return TRUE; } |