Shutdown now correctly handles connection

configure.ac

@@ -42,7 +42,7 @@ PKG_CHECK_MODULES([DEPS], [openssl glib-2.0 libcurl])
 PKG_CHECK_MODULES([NOTIFY], [libnotify], [], 
     [AC_MSG_NOTICE([libnotify module not found])])
 
-AM_CFLAGS="-O3 -Wall -W -Wconversion -Wshadow -Wcast-qual -Wwrite-strings "
+AM_CFLAGS="-Wall "
 AM_CFLAGS="$AM_CFLAGS -lstrophe -lxml2 -lexpat -lncurses -lcurl -lresolv "
 AM_CFLAGS="$AM_CFLAGS $DEPS_LIBS $NOTIFY_LIBS"
 

runvalgrind.sh

@@ -1,4 +1,4 @@
 #!/bin/sh
 rm -f valgrind.out
 #valgrind --log-file=valgrind.out --leak-check=full --track-origins=yes --show-reachable=yes ./profanity
-valgrind --log-file=valgrind.out --leak-check=full --track-origins=yes ./profanity
+valgrind --log-file=valgrind.out --leak-check=full --track-origins=yes ./profanity -l DEBUG

src/chat_log.c

@@ -87,7 +87,6 @@ chat_log_chat(const gchar * const login, gchar *other,
 void
 chat_log_close(void)
 {
-    log_info("Closing down chat logs");
     g_hash_table_foreach(logs, (GHFunc) _close_file, NULL);
     g_time_zone_unref(tz);
 }

src/command.c

@@ -494,8 +494,8 @@ _cmd_connect(const char * const inp, struct cmd_help_t help)
 static gboolean
 _cmd_quit(const char * const inp, struct cmd_help_t help)
 {
-    log_info("Profanity is shutting down.");
-    profanity_shutdown_init();
+    log_info("Profanity is shutting down...");
+    exit(0);
     return FALSE;
 }
 

src/jabber.c

@@ -389,8 +389,6 @@ _jabber_conn_handler(xmpp_conn_t * const conn,
 
             jabber_conn.conn_status = JABBER_DISCONNECTED;
             jabber_conn.presence = PRESENCE_OFFLINE;
-
-            profanity_shutdown();
             
         // lost connection for unkown reason
         } else if (jabber_conn.conn_status == JABBER_CONNECTED) {

src/log.c

@@ -121,6 +121,7 @@ log_msg(log_level_t level, const char * const area, const char * const msg)
         g_date_time_unref(dt);
     
         fflush(logp);
+        g_free(date_fmt);
     }
 }
 

src/preferences.c

@@ -117,7 +117,6 @@ prefs_load(void)
 void
 prefs_close(void)
 {
-    log_info("Closing down preferences");
     g_key_file_free(prefs);
 }
 

src/profanity.c

@@ -68,7 +68,6 @@ profanity_run(void)
         inp[size++] = '\0';
         cmd_result = _process_input(inp);
     }
-
 }
 
 void
@@ -91,11 +90,10 @@ profanity_init(const int disable_tls, char *log_level)
 void
 profanity_shutdown_init(void)
 {
-    log_info("Profanity is shutting down.");
     gboolean wait_response = jabber_disconnect();
 
     if (wait_response) {
-        while (TRUE) {
+        while (jabber_connection_status() == JABBER_DISCONNECTING) {
             jabber_process_events();
         }
     }
@@ -109,7 +107,6 @@ profanity_shutdown(void)
     gui_close();                                                          
     chat_log_close();                                                    
     prefs_close();                                                       
-    log_info("Shutdown complete");                                       
     log_close();   
 }
 

src/windows.c

@@ -141,7 +141,6 @@ gui_refresh(void)
 void
 gui_close(void)
 {
-    log_info("Closing UI");
     endwin();
 }
 
@@ -799,6 +798,7 @@ _win_show_time(WINDOW *win)
     gchar *date_fmt = g_date_time_format(time, "%H:%M");
     wprintw(win, "%s - ", date_fmt);
     g_date_time_unref(time);
+    g_free(date_fmt);
 }
 
 static void