diff options
author | Hans de Goede <hdegoede@redhat.com> | 2013-06-19 16:47:41 +0200 |
---|---|---|
committer | Hans de Goede <hdegoede@redhat.com> | 2013-09-17 14:57:41 +0200 |
commit | edaf892e8fdcf2ceb956e8dc0a41c928c3fff13f (patch) | |
tree | ba376485d8dc5b44615696c44996281366d16918 | |
parent | e0bf5c088e7d438b3f64cd9ceeee639a03bf484a (diff) |
cheese-preferences: Simplify remove_camera_device
Now that we cache the device-node for the active camera, remove_camera_device
and its callers can be simplified.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
-rw-r--r-- | src/cheese-preferences.vala | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/src/cheese-preferences.vala b/src/cheese-preferences.vala index c46a1a5..6e0bbd5 100644 --- a/src/cheese-preferences.vala +++ b/src/cheese-preferences.vala @@ -506,12 +506,6 @@ public class Cheese.PreferencesDialog : GLib.Object TreeIter iter; camera_model.get_iter_first (out iter); - // Combobox active element. - TreeIter active_iter; - Cheese.CameraDevice active_device; - source_combo.get_active_iter (out active_iter); - camera_model.get (active_iter, 1, out active_device, -1); - // Find which device was removed. bool device_removed = false; devices.foreach ((device) => @@ -523,7 +517,7 @@ public class Cheese.PreferencesDialog : GLib.Object // Found the device that was removed. if (strcmp (old_device.device_node, new_device.device_node) != 0) { - remove_camera_device (iter, new_device, active_device); + remove_camera_device (iter, new_device); device_removed = true; // Remember, this is from the anonymous function! return; @@ -536,7 +530,7 @@ public class Cheese.PreferencesDialog : GLib.Object { Cheese.CameraDevice old_device; camera_model.get (iter, 1, out old_device, -1); - remove_camera_device (iter, old_device, active_device); + remove_camera_device (iter, old_device); } } } @@ -571,16 +565,15 @@ public class Cheese.PreferencesDialog : GLib.Object * Remove the supplied camera device from the device combo box model. * * @param iter the iterator of the device to remove - * @param device_node the device to remove from the combo box model - * @param active_device_node the currently-active camera device + * @param device the device to remove from the combo box model */ - private void remove_camera_device (TreeIter iter, Cheese.CameraDevice device_node, - Cheese.CameraDevice active_device_node) + 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 (strcmp (device_node.device_node, active_device_node.device_node) == 0) + if (device.device_node == camera_device_node) { if (devices.len > 0) set_new_available_camera_device (iter); |