[libadwaita/msvc: 45/49] tests: Remove g_auto* usage




commit c5aed6377b4d7ba64e6ddfac94cf48cca17d960e
Author: Chun-wei Fan <fanchunwei src gnome org>
Date:   Mon Jul 26 19:00:02 2021 +0800

    tests: Remove g_auto* usage
    
    It is unfortunately a GCCism, so use the traditional method instead.
    Unfortunately the autocleanup compiler extensions are not standard across the
    board.

 tests/test-button-content.c |  7 ++++---
 tests/test-leaflet.c        |  4 +++-
 tests/test-status-page.c    |  6 ++++--
 tests/test-tab-view.c       | 10 +++++++---
 tests/test-window-title.c   |  6 ++++--
 5 files changed, 22 insertions(+), 11 deletions(-)
---
diff --git a/tests/test-button-content.c b/tests/test-button-content.c
index 5d5ba196..c4e97c3d 100644
--- a/tests/test-button-content.c
+++ b/tests/test-button-content.c
@@ -20,7 +20,7 @@ static void
 test_adw_button_content_icon_name (void)
 {
   AdwButtonContent *content = g_object_ref_sink (ADW_BUTTON_CONTENT (adw_button_content_new ()));
-  g_autofree char *icon_name = NULL;
+  char *icon_name = NULL;
 
   g_assert_nonnull (content);
 
@@ -40,7 +40,7 @@ test_adw_button_content_icon_name (void)
   g_object_set (content, "icon-name", "", NULL);
   g_assert_cmpstr (adw_button_content_get_icon_name (content), ==, "");
   g_assert_cmpint (notified, ==, 2);
-
+  g_free (icon_name);
   g_assert_finalize_object (content);
 }
 
@@ -48,7 +48,7 @@ static void
 test_adw_button_content_label (void)
 {
   AdwButtonContent *content = g_object_ref_sink (ADW_BUTTON_CONTENT (adw_button_content_new ()));
-  g_autofree char *label = NULL;
+  char *label;
 
   g_assert_nonnull (content);
 
@@ -69,6 +69,7 @@ test_adw_button_content_label (void)
   g_assert_cmpstr (adw_button_content_get_label (content), ==, "");
   g_assert_cmpint (notified, ==, 2);
 
+  g_free (label);
   g_assert_finalize_object (content);
 }
 
diff --git a/tests/test-leaflet.c b/tests/test-leaflet.c
index 39f7aeed..284bc886 100644
--- a/tests/test-leaflet.c
+++ b/tests/test-leaflet.c
@@ -12,11 +12,13 @@ assert_page_position (GtkSelectionModel *pages,
                       GtkWidget         *widget,
                       int                position)
 {
-  g_autoptr (AdwLeafletPage) page = NULL;
+  AdwLeafletPage *page = NULL;
 
   page = g_list_model_get_item (G_LIST_MODEL (pages), position);
 
   g_assert_true (widget == adw_leaflet_page_get_child (page));
+
+  g_object_unref (page);
 }
 
 
diff --git a/tests/test-status-page.c b/tests/test-status-page.c
index 7a497961..2c7f7bd0 100644
--- a/tests/test-status-page.c
+++ b/tests/test-status-page.c
@@ -46,7 +46,7 @@ static void
 test_adw_status_page_title (void)
 {
   AdwStatusPage *status_page = ADW_STATUS_PAGE (g_object_ref_sink (adw_status_page_new ()));
-  g_autofree char *title = NULL;
+  char *title = NULL;
 
   g_assert_nonnull (status_page);
 
@@ -67,6 +67,7 @@ test_adw_status_page_title (void)
   g_assert_cmpstr (adw_status_page_get_title (status_page), ==, "Other Title");
   g_assert_cmpint (notified, ==, 2);
 
+  g_free (title);
   g_assert_finalize_object (status_page);
 }
 
@@ -74,7 +75,7 @@ static void
 test_adw_status_page_description (void)
 {
   AdwStatusPage *status_page = ADW_STATUS_PAGE (g_object_ref_sink (adw_status_page_new ()));
-  g_autofree char *description = NULL;
+  char *description = NULL;
 
   g_assert_nonnull (status_page);
 
@@ -95,6 +96,7 @@ test_adw_status_page_description (void)
   g_assert_cmpstr (adw_status_page_get_description (status_page), ==, "Other description");
   g_assert_cmpint (notified, ==, 2);
 
+  g_free (description);
   g_assert_finalize_object (status_page);
 }
 
diff --git a/tests/test-tab-view.c b/tests/test-tab-view.c
index 70d21531..125419a6 100644
--- a/tests/test-tab-view.c
+++ b/tests/test-tab-view.c
@@ -152,6 +152,7 @@ test_adw_tab_view_n_pinned_pages (void)
   g_assert_cmpint (notified, ==, 3);
 
   g_assert_finalize_object (view);
+  g_object_unref (page);
 }
 
 static void
@@ -160,7 +161,7 @@ test_adw_tab_view_default_icon (void)
   AdwTabView *view = g_object_ref_sink (ADW_TAB_VIEW (adw_tab_view_new ()));
   GIcon *icon1 = g_themed_icon_new ("go-previous-symbolic");
   GIcon *icon2 = g_themed_icon_new ("go-next-symbolic");
-  g_autofree char *icon_str = NULL;
+  char *icon_str = NULL;
 
   g_assert_nonnull (view);
 
@@ -170,6 +171,7 @@ test_adw_tab_view_default_icon (void)
   icon_str = g_icon_to_string (adw_tab_view_get_default_icon (view));
   g_assert_cmpstr (icon_str, ==, "adw-tab-icon-missing-symbolic");
   g_assert_cmpint (notified, ==, 0);
+  g_free (icon_str);
 
   adw_tab_view_set_default_icon (view, icon1);
   g_assert_true (adw_tab_view_get_default_icon (view) == icon1);
@@ -962,7 +964,7 @@ test_adw_tab_page_title (void)
 {
   AdwTabView *view = g_object_ref_sink (ADW_TAB_VIEW (adw_tab_view_new ()));
   AdwTabPage *page;
-  g_autofree char *title = NULL;
+  char *title = NULL;
 
   g_assert_nonnull (view);
 
@@ -983,6 +985,7 @@ test_adw_tab_page_title (void)
   g_object_set (page, "title", "Some other title", NULL);
   g_assert_cmpstr (adw_tab_page_get_title (page), ==, "Some other title");
   g_assert_cmpint (notified, ==, 2);
+  g_free (title);
 
   g_assert_finalize_object (view);
 }
@@ -992,7 +995,7 @@ test_adw_tab_page_tooltip (void)
 {
   AdwTabView *view = g_object_ref_sink (ADW_TAB_VIEW (adw_tab_view_new ()));
   AdwTabPage *page;
-  g_autofree char *tooltip = NULL;
+  char *tooltip = NULL;
 
   g_assert_nonnull (view);
 
@@ -1014,6 +1017,7 @@ test_adw_tab_page_tooltip (void)
   g_assert_cmpstr (adw_tab_page_get_tooltip (page), ==, "Some other tooltip");
   g_assert_cmpint (notified, ==, 2);
 
+  g_free (tooltip);
   g_assert_finalize_object (view);
 }
 
diff --git a/tests/test-window-title.c b/tests/test-window-title.c
index 0a5616fc..d48242cb 100644
--- a/tests/test-window-title.c
+++ b/tests/test-window-title.c
@@ -20,7 +20,7 @@ static void
 test_adw_window_title_title (void)
 {
   AdwWindowTitle *window_title = g_object_ref_sink (ADW_WINDOW_TITLE (adw_window_title_new ("Some title", 
NULL)));
-  g_autofree char *title = NULL;
+  char *title = NULL;
 
   g_assert_nonnull (window_title);
 
@@ -29,6 +29,7 @@ test_adw_window_title_title (void)
 
   g_object_get (window_title, "title", &title, NULL);
   g_assert_cmpstr (title, ==, "Some title");
+  g_free (title);
 
   adw_window_title_set_title (window_title, "Some title");
   g_assert_cmpint (notified, ==, 0);
@@ -48,7 +49,7 @@ static void
 test_adw_window_title_subtitle (void)
 {
   AdwWindowTitle *window_title = g_object_ref_sink (ADW_WINDOW_TITLE (adw_window_title_new (NULL, "Some 
subtitle")));
-  g_autofree char *subtitle = NULL;
+  char *subtitle = NULL;
 
   g_assert_nonnull (window_title);
 
@@ -57,6 +58,7 @@ test_adw_window_title_subtitle (void)
 
   g_object_get (window_title, "subtitle", &subtitle, NULL);
   g_assert_cmpstr (subtitle, ==, "Some subtitle");
+  g_free (subtitle);
 
   adw_window_title_set_subtitle (window_title, "Some subtitle");
   g_assert_cmpint (notified, ==, 0);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]