[easytag] Use g_return_val_if_fail() for more invariants
- From: David King <davidk src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [easytag] Use g_return_val_if_fail() for more invariants
- Date: Sun, 19 May 2013 15:43:59 +0000 (UTC)
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]