[gnome-db] PATCH: Fix return value for gda_drop_table()
- From: Phil Longstaff <plongstaff rogers com>
- To: gnome-db list <gnome-db-list gnome org>
- Subject: [gnome-db] PATCH: Fix return value for gda_drop_table()
- Date: Sun, 09 Mar 2008 16:50:00 -0400
The convenience function gda_drop_table() returns the wrong value.
First, there are braces missing on an if-then-else, so that even if the
operation is successful, the error path is taken. Secondly,
gda_server_provider_perform_operation() returns TRUE if the operation
has no error, so the sense of the test needs to be reversed.
I've attached a patch to the libgda 3.0.2 source to fix these problems.
Phil
--- ../../libgda-3.0.2/libgda/gda-init.c 2008-01-28 11:17:26.000000000 -0500
+++ ./gda-init.c 2008-03-09 16:41:01.000000000 -0400
@@ -442,11 +442,12 @@
retval = FALSE;
}
else {
- if (gda_server_provider_perform_operation (server, cnn, op, error))
+ if (!gda_server_provider_perform_operation (server, cnn, op, error)) {
/* error */
g_object_unref (op);
xmlFreeDoc(parameters);
- return FALSE;
+ return FALSE;
+ }
}
g_object_unref (op);
xmlFreeDoc(parameters);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]