[balsa] imap-handle: Drop the last redundant clear



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]