Attachment 240891 Details for Bug 309251 – bug_309251_gparted_crashes_after_re-scanning_devices.patch

[patch] bug_309251_gparted_crashes_after_re-scanning_devices.patch

bug_309251_gparted_crashes_after_re-scanning_devices.patch (text/plain), 1.58 KB, created by Jim Hayward on 2007-10-28 15:36:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jim Hayward

Created: 2007-10-28 15:36:02 UTC

Size: 1.58 KB

patch

obsolete

>diff -ru gparted-0.3.3-old/include/Win_GParted.h gparted-0.3.3/include/Win_GParted.h
>--- gparted-0.3.3-old/include/Win_GParted.h	2006-11-25 06:56:30.000000000 -0800
>+++ gparted-0.3.3/include/Win_GParted.h	2007-10-28 07:27:34.000000000 -0700
>@@ -233,6 +233,7 @@
> 	unsigned short new_count;//new_count keeps track of the new created partitions
> 	FS fs ;
> 	bool OPERATIONSLIST_OPEN ;
>+	sigc::connection combo_devices_signal_changed_id ;
> 									
> 	GParted_Core gparted_core ;
> 	std::vector<Gtk::Label *> device_info ;
>diff -ru gparted-0.3.3-old/src/Win_GParted.cc gparted-0.3.3/src/Win_GParted.cc
>--- gparted-0.3.3-old/src/Win_GParted.cc	2006-12-01 05:14:23.000000000 -0800
>+++ gparted-0.3.3/src/Win_GParted.cc	2007-10-28 07:31:24.000000000 -0700
>@@ -260,7 +260,8 @@
> 	combo_devices .pack_start( treeview_devices_columns .device ) ;
> 	combo_devices .pack_start( treeview_devices_columns .size, false ) ;
> 	
>-	combo_devices .signal_changed() .connect( sigc::mem_fun(*this, &Win_GParted::combo_devices_changed) );
>+	combo_devices_signal_changed_id =
>+		combo_devices .signal_changed() .connect( sigc::mem_fun(*this, &Win_GParted::combo_devices_changed) );
> 
> 	hbox_toolbar .pack_start( combo_devices, Gtk::PACK_SHRINK ) ;
> }
>@@ -502,6 +503,7 @@
> 
> void Win_GParted::refresh_combo_devices()
> {
>+	combo_devices_signal_changed_id .block() ;
> 	liststore_devices ->clear() ;
> 	
> 	menu = manage( new Gtk::Menu() ) ;
>@@ -538,6 +540,7 @@
> 		menubar_main .items()[ 0 ] .get_submenu() ->items()[ 1 ] .set_submenu( *menu ) ;
> 	}
> 	
>+	combo_devices_signal_changed_id .unblock() ;
> 	combo_devices .set_active( current_device ) ;
> }
>

Actions: View | Diff

Attachments on bug 309251: 240891