From fe6586e923507bff6f15ae160ace4f8d40ffc573 Mon Sep 17 00:00:00 2001
From: Marcel Holtmann <marcel@holtmann.org>
Date: Fri, 15 May 2009 14:34:54 -0700
Subject: [PATCH] Add descriptions to all D-Bus error messages

---
 src/error.c |   19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/src/error.c b/src/error.c
index 38e90d9..844be33 100644
--- a/src/error.c
+++ b/src/error.c
@@ -41,56 +41,55 @@ DBusMessage *__connman_error_failed(DBusMessage *msg, int errnum)
 DBusMessage *__connman_error_invalid_arguments(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-						".InvalidArguments", NULL);
+				".InvalidArguments", "Invalid arguments");
 }
 
 DBusMessage *__connman_error_permission_denied(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-						".PermissionDenied", NULL);
+				".PermissionDenied", "Permission denied");
 }
 
 DBusMessage *__connman_error_not_supported(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-						".NotSupported", NULL);
+					".NotSupported", "Not supported");
 }
 
 DBusMessage *__connman_error_not_implemented(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-						".NotImplemented", NULL);
+					".NotImplemented", "Not implemented");
 }
 
 DBusMessage *__connman_error_no_carrier(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-							".NoCarrier", NULL);
+						".NoCarrier", "No carrier");
 }
 
 DBusMessage *__connman_error_in_progress(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-							".InProgress", NULL);
-
+						".InProgress", "In progress");
 }
 
 DBusMessage *__connman_error_already_connected(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-						".AlreadyConnected", NULL);
+				".AlreadyConnected", "Already connected");
 
 }
 
 DBusMessage *__connman_error_operation_aborted(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-						"OperationAborted", NULL);
+				"OperationAborted", "Operation aborted");
 }
 
 DBusMessage *__connman_error_operation_timeout(DBusMessage *msg)
 {
 	return g_dbus_create_error(msg, CONNMAN_ERROR_INTERFACE
-						"OperationTimeout", NULL);
+				"OperationTimeout", "Operation timeout");
 }
 
-- 
1.7.9.5