string-helpers.c: add string_empty()

include/common/string-helpers.h

@@ -1,6 +1,13 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 #ifndef LABWC_STRING_HELPERS_H
 #define LABWC_STRING_HELPERS_H
+#include <stdbool.h>
+
+/**
+ * string_null_or_empty() - Check if string is NULL or empty
+ * @s: string to check
+ */
+bool string_null_or_empty(const char *s);
 
 /**
  * trim_last_field() - Trim last field of string splitting on provided delim

src/button/button-png.c

@@ -12,6 +12,7 @@
 #include "buffer.h"
 #include "button/button-png.h"
 #include "button/common.h"
+#include "common/string-helpers.h"
 #include "labwc.h"
 
 /*
@@ -49,7 +50,7 @@ button_png_load(const char *button_name, struct lab_data_buffer **buffer)
 		wlr_buffer_drop(&(*buffer)->base);
 		*buffer = NULL;
 	}
-	if (!button_name || !*button_name) {
+	if (string_null_or_empty(button_name)) {
 		return;
 	}
 

src/button/button-svg.c

@@ -12,6 +12,7 @@
 #include "buffer.h"
 #include "button/button-svg.h"
 #include "button/common.h"
+#include "common/string-helpers.h"
 #include "labwc.h"
 
 void
@@ -22,7 +23,7 @@ button_svg_load(const char *button_name, struct lab_data_buffer **buffer,
 		wlr_buffer_drop(&(*buffer)->base);
 		*buffer = NULL;
 	}
-	if (!button_name || !*button_name) {
+	if (string_null_or_empty(button_name)) {
 		return;
 	}
 

src/button/button-xbm.c

@@ -17,6 +17,7 @@
 #include "button/common.h"
 #include "common/grab-file.h"
 #include "common/mem.h"
+#include "common/string-helpers.h"
 #include "buffer.h"
 
 enum token_type {
@@ -279,7 +280,7 @@ button_xbm_load(const char *button_name, struct lab_data_buffer **buffer,
 		wlr_buffer_drop(&(*buffer)->base);
 		*buffer = NULL;
 	}
-	if (!button_name || !*button_name) {
+	if (string_null_or_empty(button_name)) {
 		return;
 	}
 	color = argb32(rgba);

src/common/font.c

@@ -7,6 +7,7 @@
 #include <wlr/util/log.h>
 #include "common/font.h"
 #include "common/graphic-helpers.h"
+#include "common/string-helpers.h"
 #include "labwc.h"
 #include "buffer.h"
 
@@ -84,7 +85,7 @@ font_buffer_create(struct lab_data_buffer **buffer, int max_width,
 		max_width = 2;
 	}
 
-	if (!text || !*text) {
+	if (string_null_or_empty(text)) {
 		return;
 	}
 

src/common/string-helpers.c

@@ -6,6 +6,12 @@
 #include "common/mem.h"
 #include "common/string-helpers.h"
 
+bool
+string_null_or_empty(const char *s)
+{
+	return !s || !*s;
+}
+
 void
 trim_last_field(char *buf, char delim)
 {

src/config/libinput.c

@@ -4,6 +4,7 @@
 
 #include "common/mem.h"
 #include "common/list.h"
+#include "common/string-helpers.h"
 #include "config/libinput.h"
 #include "config/rcxml.h"
 
@@ -27,7 +28,7 @@ libinput_category_init(struct libinput_category *l)
 enum lab_libinput_device_type
 get_device_type(const char *s)
 {
-	if (!s || !*s) {
+	if (string_null_or_empty(s)) {
 		return LAB_LIBINPUT_DEVICE_NONE;
 	}
 	if (!strcasecmp(s, "default")) {

src/config/session.c

@@ -14,16 +14,10 @@
 #include "config/session.h"
 #include "labwc.h"
 
-static bool
-string_empty(const char *s)
-{
-	return !s || !*s;
-}
-
 static void
 process_line(char *line)
 {
-	if (string_empty(line) || line[0] == '#') {
+	if (string_null_or_empty(line) || line[0] == '#') {
 		return;
 	}
 	char *key = NULL;
@@ -39,7 +33,7 @@ process_line(char *line)
 	buf_add(&value, string_strip(++p));
 	buf_expand_shell_variables(&value);
 	buf_expand_tilde(&value);
-	if (string_empty(key) || !value.len) {
+	if (string_null_or_empty(key) || !value.len) {
 		goto error;
 	}
 	setenv(key, value.buf, 1);

src/ssd/ssd_titlebar.c

@@ -7,6 +7,7 @@
 #include "common/mem.h"
 #include "common/scaled_font_buffer.h"
 #include "common/scene-helpers.h"
+#include "common/string-helpers.h"
 #include "labwc.h"
 #include "node.h"
 #include "ssd-internal.h"
@@ -325,7 +326,7 @@ ssd_update_title(struct ssd *ssd)
 
 	struct view *view = ssd->view;
 	char *title = (char *)view_get_string_prop(view, "title");
-	if (!title || !*title) {
+	if (string_null_or_empty(title)) {
 		return;
 	}