prevent possible application crashes. It's useful
also for detecting mail operations with invalid
status and error handling */
- if (modest_mail_operation_get_error (mail_op) != NULL)
+ if (modest_mail_operation_get_error (mail_op) != NULL) {
modest_mail_operation_execute_error_handler (mail_op);
- else {
+ } else {
if (status == MODEST_MAIL_OPERATION_STATUS_CANCELED)
g_warning ("%s: operation canceled \n", __FUNCTION__);
else
static void
on_cancel_all_foreach (gpointer op, gpointer list)
{
+ g_return_if_fail (list);
*((GSList**)list) = g_slist_prepend (*((GSList**)list), MODEST_MAIL_OPERATION (op));
}
modest_mail_operation_queue_cancel_all (ModestMailOperationQueue *self)
{
ModestMailOperationQueuePrivate *priv;
- GSList* operations_to_cancel;
- GSList* cur;
+ GSList* operations_to_cancel = NULL;
+ GSList* cur = NULL;
g_return_if_fail (MODEST_IS_MAIL_OPERATION_QUEUE (self));
g_mutex_lock (priv->queue_lock);
g_queue_foreach (priv->op_queue, (GFunc) on_cancel_all_foreach, &operations_to_cancel);
g_mutex_unlock (priv->queue_lock);
-
+
/* TODO: Reverse the list, to remove operations in order? */
for(cur = operations_to_cancel; cur != NULL; cur = cur->next) {