toxblh/podcasts
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Prefs: Use weak refs to avoid refference cycle.

Browse Source
This commit is contained in:
Jordan Petridis 2018-07-25 04:25:40 +03:00
parent 5f7c822deb
commit aaca6a6704
No known key found for this signature in database
GPG Key ID: CEABAD9F5683B9A6
1 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
podcasts-gtk/src/prefs.rs
View File

@ -68,14 +68,21 @@ impl Prefs {
self.cleanup_type.pack_start(&renderer, true); self.cleanup_type.pack_start(&renderer, true);
self.cleanup_type.add_attribute(&renderer, "text", 0); self.cleanup_type.add_attribute(&renderer, "text", 0);
self.cleanup_type.set_active(cleanup_pos); self.cleanup_type.set_active(cleanup_pos);
self.cleanup_type
.connect_changed(clone!(settings, store => move |combo| { let weak_settings = settings.downgrade();
if let Some(ref treeiter) = combo.get_active_iter() { let weak_store = store.downgrade();
// Totally the prettiest stairway you ever saw
self.cleanup_type.connect_changed(move |combo| {
if let Some(ref treeiter) = combo.get_active_iter() {
if let Some(store) = weak_store.upgrade() {
if let Some(s) = store.get_value(treeiter, 0).get::<&str>() { if let Some(s) = store.get_value(treeiter, 0).get::<&str>() {
settings.set_string("cleanup-age-period", &s.to_lowercase()); if let Some(settings) = weak_settings.upgrade() {
settings.set_string("cleanup-age-period", &s.to_lowercase());
}
} }
}; }
})); };
});
} }
pub fn show(&self, parent: &gtk::ApplicationWindow) { pub fn show(&self, parent: &gtk::ApplicationWindow) {