[easytag] Use g_return_val_if_fail() for more invariants



commit f7c0cc8ccebcd9c98cc7a1fcfb863c63f859142f
Author: David King <amigadave amigadave com>
Date:   Sun May 19 16:36:24 2013 +0100

    Use g_return_val_if_fail() for more invariants
    
    Additionally, refactor Cddb_Get_Pixbuf_From_Server_Name() to avoid a
    cppcheck warning about duplicate if and else branches.

 src/cddb.c    |   17 +++++------------
 src/picture.c |    4 ++--
 2 files changed, 7 insertions(+), 14 deletions(-)
---
diff --git a/src/cddb.c b/src/cddb.c
index 2564b4d..b1601cd 100644
--- a/src/cddb.c
+++ b/src/cddb.c
@@ -1506,12 +1506,8 @@ Cddb_Open_Connection (const gchar *host, gint port)
     gint               optval = 1;
     gchar             *msg;
 
-
-    if (!CddbWindow)
-        return 0;
-
-    if (!host || port <= 0)
-        return 0;
+    g_return_val_if_fail (CddbWindow != NULL, 0);
+    g_return_val_if_fail (host != NULL && port > 0, 0);
 
     msg = g_strdup_printf(_("Resolving host '%s'…"),host);
     gtk_statusbar_push(GTK_STATUSBAR(CddbStatusBar),CddbStatusBarContext,msg);
@@ -4091,17 +4087,14 @@ Cddb_Get_Id3_Genre_From_Cddb_Genre (const gchar *cddb_genre)
 static GdkPixbuf *
 Cddb_Get_Pixbuf_From_Server_Name (const gchar *server_name)
 {
-    if (!server_name)
-        return NULL;
-    else if (strstr(server_name,"freedb.org"))
+    g_return_val_if_fail (server_name != NULL, NULL);
+
+    if (strstr (server_name, "freedb.org"))
         return gdk_pixbuf_new_from_xpm_data(freedb_xpm);
     else if (strstr(server_name,"gnudb.org"))
         return gdk_pixbuf_new_from_xpm_data(gnudb_xpm);
     else if (strstr(server_name,"musicbrainz.org"))
         return gdk_pixbuf_new_from_xpm_data(musicbrainz_xpm);
-    else if (strstr(server_name,"/"))
-        //return gdk_pixbuf_new_from_xpm_data(closed_folder_xpm);
-        return NULL;
     else
         return NULL;
 }
diff --git a/src/picture.c b/src/picture.c
index ec83be4..b559c41 100644
--- a/src/picture.c
+++ b/src/picture.c
@@ -1061,8 +1061,8 @@ Picture *Picture_Copy_One (const Picture *pic)
 {
     Picture *pic2;
 
-    if (!pic)
-        return 0;
+    g_return_val_if_fail (pic != NULL, NULL);
+
     pic2 = Picture_Allocate();
     pic2->type = pic->type;
     pic2->width  = pic->width;


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