utils: Uniform the no-devices error messages removing duplicated checks

Use error messages to be consistent, and avoid checking for a returned
value when dbus-glib function to fetch it returned false, as it's
implicit that we had a failure.

Otherwise if didn't fail we are sure that we got the requested value.
This commit is contained in:
Marco Trevisan (Treviño)
2020-04-03 03:29:12 +02:00
parent 091f373109
commit ecc02cb588
4 changed files with 6 additions and 16 deletions

View File

@ -89,12 +89,12 @@ static void process_devices(char **argv)
guint i;
if (!net_reactivated_Fprint_Manager_get_devices(manager, &devices, &error)) {
g_print("list_devices failed: %s\n", error->message);
g_print("Impossible to get devices: %s\n", error->message);
exit (1);
}
if (devices->len == 0) {
g_print("No devices found\n");
g_print("No devices available\n");
exit(1);
}

View File

@ -57,12 +57,7 @@ static DBusGProxy *open_device(const char *username)
DBusGProxy *dev;
if (!net_reactivated_Fprint_Manager_get_default_device(manager, &path, &error)) {
g_print("list_devices failed: %s\n", error->message);
exit (1);
}
if (path == NULL) {
g_print("No devices found\n");
g_print("Impossible to enroll: %s\n", error->message);
exit (1);
}

View File

@ -94,12 +94,12 @@ static void process_devices(char **argv)
guint i;
if (!net_reactivated_Fprint_Manager_get_devices(manager, &devices, &error)) {
g_print("list_devices failed: %s\n", error->message);
g_print("Impossible to get devices: %s\n", error->message);
exit (1);
}
if (devices->len == 0) {
g_print("No devices found\n");
g_print("No devices available\n");
exit(1);
}

View File

@ -54,12 +54,7 @@ static DBusGProxy *open_device(const char *username)
DBusGProxy *dev;
if (!net_reactivated_Fprint_Manager_get_default_device(manager, &path, &error)) {
g_print("list_devices failed: %s\n", error->message);
exit (1);
}
if (path == NULL) {
g_print("No devices found\n");
g_print("Impossible to verify: %s\n", error->message);
exit (1);
}