[nautilus/freedesktop-dbus] Don't use nested if()s when a single one will do



commit 9868b796ec9d4761512d0a948bf0d6438be6702c
Author: Federico Mena Quintero <federico gnome org>
Date:   Tue Dec 13 17:58:41 2011 -0600

    Don't use nested if()s when a single one will do

 src/nautilus-properties-window.c |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)
---
diff --git a/src/nautilus-properties-window.c b/src/nautilus-properties-window.c
index b715567..5d952fb 100644
--- a/src/nautilus-properties-window.c
+++ b/src/nautilus-properties-window.c
@@ -4969,11 +4969,10 @@ remove_pending (StartupData *startup_data,
 		eel_timed_wait_stop 
 			(cancel_create_properties_window_callback, startup_data);
 	}
-	if (cancel_destroy_handler) {
-		if (startup_data->parent_widget)
-			g_signal_handlers_disconnect_by_func (startup_data->parent_widget,
-							      G_CALLBACK (parent_widget_destroyed_callback),
-							      startup_data);
+	if (cancel_destroy_handler && startup_data->parent_widget) {
+		g_signal_handlers_disconnect_by_func (startup_data->parent_widget,
+						      G_CALLBACK (parent_widget_destroyed_callback),
+						      startup_data);
 	}
 
 	g_hash_table_remove (pending_lists, startup_data->pending_key);



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]