summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author7 <NotZed@Ximian.com>2001-09-27 23:24:01 +0000
committerMichael Zucci <zucchi@src.gnome.org>2001-09-27 23:24:01 +0000
commitebf9709024e5c4aeb51d0e7f06e9e10f59473829 (patch)
treef2e2fc25504da4b40f64061904c81b16090ea6c5
parentfc1a171c33dd49196cbe76444c50c03a294db117 (diff)
downloadevolution-data-server-ebf9709024e5c4aeb51d0e7f06e9e10f59473829.tar.gz
Dont re-register the connect_op if we got it by calling
2001-09-27 <NotZed@Ximian.com> * camel-service.c (camel_service_connect): Dont re-register the connect_op if we got it by calling 'operation_registered', which returns an already-registered one. (camel_service_disconnect): Likewise here. This removes all the re-registered warnings.
-rw-r--r--camel/ChangeLog6
-rw-r--r--camel/camel-service.c10
2 files changed, 12 insertions, 4 deletions
diff --git a/camel/ChangeLog b/camel/ChangeLog
index b3eccf05f..8c77bf23d 100644
--- a/camel/ChangeLog
+++ b/camel/ChangeLog
@@ -1,5 +1,11 @@
2001-09-27 <NotZed@Ximian.com>
+ * camel-service.c (camel_service_connect): Dont re-register the
+ connect_op if we got it by calling 'operation_registered', which
+ returns an already-registered one.
+ (camel_service_disconnect): Likewise here.
+ This removes all the re-registered warnings.
+
* camel-object.c (camel_object_unref): Fill the finalised data
with nonsense, rather than 0's - more easily detect bad data.
(camel_object_new): Clear instance data when we retrieve it off
diff --git a/camel/camel-service.c b/camel/camel-service.c
index cc254e1f2..fed894500 100644
--- a/camel/camel-service.c
+++ b/camel/camel-service.c
@@ -247,9 +247,10 @@ camel_service_connect (CamelService *service, CamelException *ex)
*/
CAMEL_SERVICE_LOCK (service, connect_op_lock);
service->connect_op = camel_operation_registered ();
- if (!service->connect_op)
+ if (!service->connect_op) {
service->connect_op = camel_operation_new (NULL, NULL);
- camel_operation_register (service->connect_op);
+ camel_operation_register (service->connect_op);
+ }
CAMEL_SERVICE_UNLOCK (service, connect_op_lock);
service->status = CAMEL_SERVICE_CONNECTING;
@@ -301,9 +302,10 @@ camel_service_disconnect (CamelService *service, gboolean clean,
if (service->status == CAMEL_SERVICE_CONNECTED) {
CAMEL_SERVICE_LOCK (service, connect_op_lock);
service->connect_op = camel_operation_registered ();
- if (!service->connect_op)
+ if (!service->connect_op) {
service->connect_op = camel_operation_new (NULL, NULL);
- camel_operation_register (service->connect_op);
+ camel_operation_register (service->connect_op);
+ }
CAMEL_SERVICE_UNLOCK (service, connect_op_lock);
service->status = CAMEL_SERVICE_DISCONNECTING;