From 0c841182b61efe48e98c6d2dea1f8c7ab7f464ef Mon Sep 17 00:00:00 2001 From: Hans de Goede Date: Wed, 19 Jun 2013 17:29:28 +0200 Subject: [PATCH 10/11] cheese-preferences: Cleanly handle going from 1 -> 0 devices Cleanly handle going from 1 -> 0 devices and going from 1 -> 0 -> 1 devices. Signed-off-by: Hans de Goede --- src/cheese-preferences.vala | 26 +++++++++++++++----------- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/src/cheese-preferences.vala b/src/cheese-preferences.vala index 6e0bbd5..fc72435 100644 --- a/src/cheese-preferences.vala +++ b/src/cheese-preferences.vala @@ -278,13 +278,13 @@ public class Cheese.PreferencesDialog : GLib.Object [CCode (instance_pos = -1)] public void on_source_change (Gtk.ComboBox combo) { - // TODO: Handle going from 1 to 0 devices, cleanly! - return_if_fail (camera.num_camera_devices > 0); - TreeIter iter; Cheese.CameraDevice dev; string dev_node; + if (combo.get_active () == -1) + return; + combo.get_active_iter (out iter); combo.model.get (iter, 1, out dev); dev_node = dev.get_device_node (); @@ -570,16 +570,20 @@ public class Cheese.PreferencesDialog : GLib.Object private void remove_camera_device (TreeIter iter, Cheese.CameraDevice device) { - unowned GLib.PtrArray devices = camera.get_camera_devices (); - - // Check if the camera that we want to remove, is the active one - if (device.device_node == camera_device_node) + if (camera.num_camera_devices == 0) /* Last camera gone? */ { - if (devices.len > 0) - set_new_available_camera_device (iter); - else - this.dialog.hide(); + ListStore resolution_model = new ListStore (2, typeof (string), + typeof (Cheese.VideoFormat)); + photo_resolution_combo.model = resolution_model; + video_resolution_combo.model = resolution_model; + camera_device_node = ""; + camera_error(_("No device found")); } + else if (device.device_node == camera_device_node) + { + set_new_available_camera_device (iter); + } + camera_model.remove (iter); if (camera_model.iter_n_children (null) <= 1) -- 1.8.3.1