Remove obsolete egg-string code

Closes https://github.com/mate-desktop/mate-power-manager/pull/141

4 files changed, 30 insertions, 504 deletions

diff --git a/src/Makefile.am b/src/Makefile.am
index ddbb6ed..907ff5c 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -76,8 +76,6 @@ libgpmshared_a_SOURCES =				\
 	egg-dbus-monitor.c				\
 	egg-discrete.h					\
 	egg-discrete.c					\
-	egg-string.h					\
-	egg-string.c					\
 	egg-console-kit.h				\
 	egg-console-kit.c				\
 	gpm-common.h					\
diff --git a/src/egg-string.c b/src/egg-string.c
deleted file mode 100644
index 204e6d9..0000000
--- a/src/egg-string.c
+++ /dev/null
@@ -1,453 +0,0 @@
-/* -*- Mode: C; tab-width: 8; indent-tabs-mode: t; c-basic-offset: 8 -*-
- *
- * Copyright (C) 2007-2008 Richard Hughes <[email protected]>
- *
- * Licensed under the GNU General Public License Version 2
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
- */
-
-/**
- * SECTION:pk-common
- * @short_description: Common utility functions for PackageKit
- *
- * This file contains functions that may be useful.
- */
-
-#include "config.h"
-
-#include <stdlib.h>
-#include <stdio.h>
-
-#include <string.h>
-#include <sys/types.h>
-#include <sys/utsname.h>
-#include <sys/stat.h>
-
-#ifdef HAVE_UNISTD_H
-#include <unistd.h>
-#endif /* HAVE_UNISTD_H */
-
-#include <glib.h>
-
-#include "egg-debug.h"
-#include "egg-string.h"
-
-/**
- * egg_strtoint:
- * @text: The text the convert
- * @value: The return numeric return value
- *
- * Converts a string into a signed integer value in a safe way.
- *
- * Return value: %TRUE if the string was converted correctly
- **/
-gboolean
-egg_strtoint (const gchar *text, gint *value)
-{
-	gchar *endptr = NULL;
-	gint64 value_raw;
-
-	/* invalid */
-	if (text == NULL)
-		return FALSE;
-
-	/* parse */
-	value_raw = g_ascii_strtoll (text, &endptr, 10);
-
-	/* parsing error */
-	if (endptr == text)
-		return FALSE;
-
-	/* out of range */
-	if (value_raw > G_MAXINT || value_raw < G_MININT)
-		return FALSE;
-
-	/* cast back down to value */
-	*value = (gint) value_raw;
-	return TRUE;
-}
-
-/**
- * egg_strtouint:
- * @text: The text the convert
- * @value: The return numeric return value
- *
- * Converts a string into a unsigned integer value in a safe way.
- *
- * Return value: %TRUE if the string was converted correctly
- **/
-gboolean
-egg_strtouint (const gchar *text, guint *value)
-{
-	gchar *endptr = NULL;
-	guint64 value_raw;
-
-	/* invalid */
-	if (text == NULL)
-		return FALSE;
-
-	/* parse */
-	value_raw = g_ascii_strtoull (text, &endptr, 10);
-
-	/* parsing error */
-	if (endptr == text)
-		return FALSE;
-
-	/* out of range */
-	if (value_raw > G_MAXINT)
-		return FALSE;
-
-	/* cast back down to value */
-	*value = (guint) value_raw;
-	return TRUE;
-}
-
-/**
- * egg_strzero:
- * @text: The text to check
- *
- * This function is a much safer way of doing "if (strlen (text) == 0))"
- * as it does not rely on text being NULL terminated. It's also much
- * quicker as it only checks the first byte rather than scanning the whole
- * string just to verify it's not zero length.
- *
- * Return value: %TRUE if the string was converted correctly
- **/
-gboolean
-egg_strzero (const gchar *text)
-{
-	if (text == NULL)
-		return TRUE;
-	if (text[0] == '\0')
-		return TRUE;
-	return FALSE;
-}
-
-/**
- * egg_strlen:
- * @text: The text to check
- * @len: The maximum length of the string
- *
- * This function is a much safer way of doing strlen as it checks for NULL and
- * a stupidly long string.
- *
- * Return value: the length of the string, or len if the string is too long.
- **/
-guint
-egg_strlen (const gchar *text, guint len)
-{
-	guint i;
-
-	/* common case */
-	if (text == NULL || text[0] == '\0')
-		return 0;
-
-	/* only count up to len */
-	for (i=1; i<len; i++) {
-		if (text[i] == '\0')
-			break;
-	}
-	return i;
-}
-
-/**
- * egg_strvequal:
- * @id1: the first item of text to test
- * @id2: the second item of text to test
- *
- * This function will check to see if the GStrv arrays are string equal
- *
- * Return value: %TRUE if the arrays are the same, or are both %NULL
- **/
-gboolean
-egg_strvequal (gchar **id1, gchar **id2)
-{
-	guint i;
-	guint length1;
-	guint length2;
-
-	if (id1 == NULL && id2 == NULL)
-		return TRUE;
-
-	if (id1 == NULL || id2 == NULL) {
-		egg_debug ("GStrv compare invalid '%p' and '%p'", id1, id2);
-		return FALSE;
-	}
-
-	/* check different sizes */
-	length1 = g_strv_length (id1);
-	length2 = g_strv_length (id2);
-	if (length1 != length2)
-		return FALSE;
-
-	/* text equal each one */
-	for (i=0; i<length1; i++) {
-		if (g_strcmp0 (id1[i], id2[i]) != 0)
-			return FALSE;
-	}
-
-	return TRUE;
-}
-
-/**
- * egg_strreplace:
- * @text: The input text to make safe
- * @find: What to search for
- * @replace: What to replace with
- *
- * Replaces chars in the text with a replacement.
- * The %find and %replace variables to not have to be of the same length
- *
- * Return value: the new string (copied)
- **/
-gchar *
-egg_strreplace (const gchar *text, const gchar *find, const gchar *replace)
-{
-	gchar **array;
-	gchar *retval;
-
-	/* common case, not found */
-	if (strstr (text, find) == NULL) {
-		return g_strdup (text);
-	}
-
-	/* split apart and rejoin with new delimiter */
-	array = g_strsplit (text, find, 0);
-	retval = g_strjoinv (replace, array);
-	g_strfreev (array);
-	return retval;
-}
-
-/***************************************************************************
- ***                          MAKE CHECK TESTS                           ***
- ***************************************************************************/
-#ifdef EGG_TEST
-#include "egg-test.h"
-
-void
-egg_string_test (EggTest *test)
-{
-	gboolean ret;
-	gchar *text_safe;
-	const gchar *temp;
-	guint length;
-	gint value;
-	guint uvalue;
-	gchar **id1;
-	gchar **id2;
-
-	if (!egg_test_start (test, "EggString"))
-		return;
-
-	/************************************************************
-	 ****************    String array equal    ******************
-	 ************************************************************/
-	egg_test_title (test, "egg_strvequal same argument");
-	id1 = g_strsplit ("the quick brown fox", " ", 0);
-	if (egg_strvequal (id1, id1))
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "incorrect ret when both same");
-	g_strfreev (id1);
-
-	/************************************************************/
-	egg_test_title (test, "egg_strvequal same");
-	id1 = g_strsplit ("the quick brown fox", " ", 0);
-	id2 = g_strsplit ("the quick brown fox", " ", 0);
-	if (egg_strvequal (id1, id2))
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "incorrect ret when both same");
-	g_strfreev (id1);
-	g_strfreev (id2);
-
-	/************************************************************/
-	egg_test_title (test, "egg_strvequal different lengths");
-	id1 = g_strsplit ("the quick brown", " ", 0);
-	id2 = g_strsplit ("the quick brown fox", " ", 0);
-	if (!egg_strvequal (id1, id2))
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "incorrect ret when both same");
-	g_strfreev (id1);
-	g_strfreev (id2);
-
-	/************************************************************/
-	egg_test_title (test, "egg_strvequal different");
-	id1 = g_strsplit ("the quick brown fox", " ", 0);
-	id2 = g_strsplit ("richard hughes maintainer dude", " ", 0);
-	if (!egg_strvequal (id1, id2))
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "same when different");
-	g_strfreev (id1);
-	g_strfreev (id2);
-
-	/************************************************************
-	 ****************          Zero            ******************
-	 ************************************************************/
-	temp = NULL;
-	egg_test_title (test, "test strzero (null)");
-	ret = egg_strzero (NULL);
-	if (ret)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed null");
-
-	/************************************************************/
-	egg_test_title (test, "test strzero (null first char)");
-	ret = egg_strzero ("");
-	if (ret)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed null");
-
-	/************************************************************/
-	egg_test_title (test, "test strzero (long string)");
-	ret = egg_strzero ("Richard");
-	if (!ret)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "zero length word!");
-
-	/************************************************************/
-	egg_test_title (test, "id strcmp pass");
-	ret = (g_strcmp0 ("moo;0.0.1;i386;fedora", "moo;0.0.1;i386;fedora") == 0);
-	egg_test_assert (test, ret);
-
-	/************************************************************/
-	egg_test_title (test, "id strcmp fail");
-	ret = (g_strcmp0 ("moo;0.0.1;i386;fedora", "moo;0.0.2;i386;fedora") == 0);
-	egg_test_assert (test, !ret);
-
-	/************************************************************
-	 ****************          strlen          ******************
-	 ************************************************************/
-	egg_test_title (test, "strlen bigger");
-	length = egg_strlen ("123456789", 20);
-	if (length == 9)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed the strlen %i", length);
-
-	/************************************************************/
-	egg_test_title (test, "strlen smaller");
-	length = egg_strlen ("123456789", 5);
-	if (length == 5)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed the strlen %i", length);
-
-	/************************************************************/
-	egg_test_title (test, "strlen correct");
-	length = egg_strlen ("123456789", 9);
-	if (length == 9)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed the strlen %i", length);
-
-	/************************************************************
-	 ****************         Replace          ******************
-	 ************************************************************/
-	egg_test_title (test, "replace start");
-	text_safe = egg_strreplace ("richard\nhughes", "r", "e");
-	if (g_strcmp0 (text_safe, "eichaed\nhughes") == 0)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed the replace '%s'", text_safe);
-	g_free (text_safe);
-
-	/************************************************************/
-	egg_test_title (test, "replace none");
-	text_safe = egg_strreplace ("richard\nhughes", "dave", "e");
-	if (g_strcmp0 (text_safe, "richard\nhughes") == 0)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed the replace '%s'", text_safe);
-	g_free (text_safe);
-
-	/************************************************************/
-	egg_test_title (test, "replace end");
-	text_safe = egg_strreplace ("richard\nhughes", "s", "e");
-	if (g_strcmp0 (text_safe, "richard\nhughee") == 0)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed the replace '%s'", text_safe);
-	g_free (text_safe);
-
-	/************************************************************/
-	egg_test_title (test, "replace unicode");
-	text_safe = egg_strreplace ("richard\n- hughes", "\n- ", "\n• ");
-	if (g_strcmp0 (text_safe, "richard\n• hughes") == 0)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "failed the replace '%s'", text_safe);
-	g_free (text_safe);
-
-	/************************************************************
-	 **************        Convert numbers       ****************
-	 ************************************************************/
-	egg_test_title (test, "convert valid number");
-	ret = egg_strtoint ("234", &value);
-	if (ret && value == 234)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "value is %i", value);
-
-	/************************************************************/
-	egg_test_title (test, "convert negative valid number");
-	ret = egg_strtoint ("-234", &value);
-	if (ret && value == -234)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "value is %i", value);
-
-	/************************************************************/
-	egg_test_title (test, "don't convert invalid number");
-	ret = egg_strtoint ("dave", &value);
-	if (!ret)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "value is %i", value);
-
-	/************************************************************/
-	egg_test_title (test, "convert NULL to a number");
-	ret = egg_strtouint (NULL, &uvalue);
-	if (!ret)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "value is %i", uvalue);
-
-	/************************************************************/
-	egg_test_title (test, "convert valid uint number");
-	ret = egg_strtouint ("234", &uvalue);
-	if (ret && uvalue == 234)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "value is %i", uvalue);
-
-	/************************************************************/
-	egg_test_title (test, "convert invalid uint number");
-	ret = egg_strtouint ("-234", &uvalue);
-	if (ret == FALSE)
-		egg_test_success (test, NULL);
-	else
-		egg_test_failed (test, "value is %i", uvalue);
-
-	egg_test_end (test);
-}
-#endif
-
diff --git a/src/egg-string.h b/src/egg-string.h
deleted file mode 100644
index 6b8e097..0000000
--- a/src/egg-string.h
+++ /dev/null
@@ -1,47 +0,0 @@
-/* -*- Mode: C; tab-width: 8; indent-tabs-mode: t; c-basic-offset: 8 -*-
- *
- * Copyright (C) 2007-2008 Richard Hughes <[email protected]>
- *
- * Licensed under the GNU General Public License Version 2
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
- */
-
-#ifndef __EGG_STRING_H
-#define __EGG_STRING_H
-
-#include <glib.h>
-
-G_BEGIN_DECLS
-
-guint		 egg_strlen				(const gchar	*text,
-							 guint		 len)
-							 G_GNUC_WARN_UNUSED_RESULT;
-gboolean	 egg_strzero				(const gchar	*text)
-							 G_GNUC_WARN_UNUSED_RESULT;
-gboolean	 egg_strvequal				(gchar		**id1,
-							 gchar		**id2)
-							 G_GNUC_WARN_UNUSED_RESULT;
-gboolean	 egg_strtoint				(const gchar	*text,
-							 gint		*value);
-gboolean	 egg_strtouint				(const gchar	*text,
-							 guint		*value);
-gchar		*egg_strreplace				(const gchar	*text,
-							 const gchar	*find,
-							 const gchar	*replace);
-
-G_END_DECLS
-
-#endif /* __EGG_STRING_H */
diff --git a/src/gpm-brightness.c b/src/gpm-brightness.c
index e92b6e4..acfffc2 100644
--- a/src/gpm-brightness.c
+++ b/src/gpm-brightness.c
@@ -42,7 +42,6 @@
 
 #include "egg-discrete.h"
 #include "egg-debug.h"
-#include "egg-string.h"
 
 #include "gpm-brightness.h"
 #include "gpm-common.h"
@@ -89,6 +88,35 @@ static guint signals [LAST_SIGNAL] = { 0 };
 static gpointer gpm_brightness_object = NULL;
 
 /**
+ * gpm_brightness_helper_strtoint:
+ * @text: The text to be converted
+ * @value: The return numeric return value
+ *
+ * Convert a string to a signed integer value in a safe way.
+ *
+ * Return value: %TRUE if the string was converted correctly
+ **/
+static gboolean
+gpm_brightness_helper_strtoint (const gchar *text, gint *value)
+{
+	gchar *endptr = NULL;
+	gint64 value_raw;
+
+	if (text == NULL)
+		return FALSE;
+
+	value_raw = g_ascii_strtoll (text, &endptr, 10);
+
+	if (endptr == text)
+		return FALSE;
+	if (errno == ERANGE || value_raw > G_MAXINT || value_raw < G_MININT)
+		return FALSE;
+
+	*value = (gint) value_raw;
+	return TRUE;
+}
+
+/**
  * gpm_brightness_helper_get_value:
  **/
 static gint
@@ -113,7 +141,7 @@ gpm_brightness_helper_get_value (const gchar *argument)
 	egg_debug ("executing %s retval: %i", command, exit_status);
 
 	/* parse for a number */
-	ret = egg_strtoint (stdout_data, &value);
+	ret = gpm_brightness_helper_strtoint (stdout_data, &value);
 	if (!ret)
 		goto out;
 out: