[gnome-software/wip/kalev/issue459: 2/2] flatpak: Add asserts when building a ref
- From: Kalev Lember <klember src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-software/wip/kalev/issue459: 2/2] flatpak: Add asserts when building a ref
- Date: Sun, 23 Sep 2018 07:50:24 +0000 (UTC)
commit ce5ff567e57c0bd8a753790d26618e6950ff8144
Author: Kalev Lember <klember redhat com>
Date: Sun Sep 23 09:43:10 2018 +0200
flatpak: Add asserts when building a ref
Having these would have made debugging
https://gitlab.gnome.org/GNOME/gnome-software/issues/459 much easier and
immediately obvious what's going on.
plugins/flatpak/gs-flatpak-app.c | 18 ++++++++++++++----
1 file changed, 14 insertions(+), 4 deletions(-)
---
diff --git a/plugins/flatpak/gs-flatpak-app.c b/plugins/flatpak/gs-flatpak-app.c
index 071877e0..a71b0eed 100644
--- a/plugins/flatpak/gs-flatpak-app.c
+++ b/plugins/flatpak/gs-flatpak-app.c
@@ -105,11 +105,21 @@ gs_flatpak_app_get_repo_url (GsApp *app)
gchar *
gs_flatpak_app_get_ref_display (GsApp *app)
{
+ const gchar *ref_kind_as_str = gs_flatpak_app_get_ref_kind_as_str (app);
+ const gchar *ref_name = gs_flatpak_app_get_ref_name (app);
+ const gchar *ref_arch = gs_flatpak_app_get_ref_arch (app);
+ const gchar *ref_branch = gs_flatpak_app_get_ref_branch (app);
+
+ g_assert (ref_kind_as_str != NULL);
+ g_assert (ref_name != NULL);
+ g_assert (ref_arch != NULL);
+ g_assert (ref_branch != NULL);
+
return g_strdup_printf ("%s/%s/%s/%s",
- gs_flatpak_app_get_ref_kind_as_str (app),
- gs_flatpak_app_get_ref_name (app),
- gs_flatpak_app_get_ref_arch (app),
- gs_flatpak_app_get_ref_branch (app));
+ ref_kind_as_str,
+ ref_name,
+ ref_arch,
+ ref_branch);
}
void
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]