[balsa] imap-handle: Drop the last redundant clear
- From: Peter Bloomfield <peterb src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [balsa] imap-handle: Drop the last redundant clear
- Date: Thu, 22 Aug 2019 22:46:22 +0000 (UTC)
commit bf8bf6fe8a07c198eaf0a961842f962e874b34d2
Author: Peter Bloomfield <PeterBloomfield bellsouth net>
Date: Thu Aug 22 18:44:33 2019 -0400
imap-handle: Drop the last redundant clear
The finalize method is called only once, so clearing pointers is
unnecessary.
libbalsa/imap/imap-handle.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
---
diff --git a/libbalsa/imap/imap-handle.c b/libbalsa/imap/imap-handle.c
index f63219a23..343b8eb17 100644
--- a/libbalsa/imap/imap-handle.c
+++ b/libbalsa/imap/imap-handle.c
@@ -862,10 +862,9 @@ static void
imap_mbox_handle_finalize(GObject* gobject)
{
ImapMboxHandle* handle = IMAP_MBOX_HANDLE(gobject);
- g_return_if_fail(handle);
g_mutex_lock(&handle->mutex);
- if(handle->state != IMHS_DISCONNECTED) {
+ if (handle->state != IMHS_DISCONNECTED) {
handle->doing_logout = TRUE;
imap_cmd_exec(handle, "LOGOUT");
}
@@ -876,7 +875,7 @@ imap_mbox_handle_finalize(GObject* gobject)
g_list_foreach(handle->cmd_info, (GFunc)g_free, NULL);
g_list_free(handle->cmd_info);
- g_hash_table_destroy(handle->status_resps); handle->status_resps = NULL;
+ g_hash_table_destroy(handle->status_resps);
if (handle->thread_root != NULL)
g_node_destroy(handle->thread_root);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]