2007-06-03 Armin Burgmeier <armin@openismus.com>
authorArmin Burgmeier <armin@openismus.com>
Sun, 3 Jun 2007 14:07:12 +0000 (14:07 +0000)
committerArmin Burgmeier <armin@openismus.com>
Sun, 3 Jun 2007 14:07:12 +0000 (14:07 +0000)
commit75bc955b9e9eb099e08799367eceb4e4aadcf158
treeea2cf570c7202ba29109fa19045dce15ebdb8ca8
parent62f7397e7b5b8b70f67b67800527916aa4016f4b
2007-06-03  Armin Burgmeier  <armin@openismus.com>

* src/modest-mail-operation.c:
(modest_mail_operation_get_msgs_full): Set priv->account also when
only retrieving a single message. This avoids a crash when canceling
the operation.

(modest_mail_operation_notify_end): Do not explicitely remove the
operation from the queue but only emit the progress_changed signal.
The queue itself listens to that signal to remove the operation when
it is finished. This reduces coupling and removes a potential deadlock
when modest_mail_operation_notify_end() is called from the queue
itself. If that is a problem performance-wise, I would suggest to
implement a status-changed signal or something.

* src/modest-mail-operation-queue.c: Make sure to never call a
function from another file while the queue is locked. This (hopefully)
fixes another deadlock when exiting the application while retrieving
mail.

pmo-trunk-r2056
ChangeLog2
src/modest-mail-operation-queue.c
src/modest-mail-operation.c
src/modest-mail-operation.h