From 583794a3ec88681cd6146b50acf6383c44a9b401 Mon Sep 17 00:00:00 2001 From: Christopher Davis Date: Thu, 24 Nov 2022 12:26:04 -0500 Subject: [PATCH] general: Use glib::Object::builder for objects w/ props Builders are more idiomatic rust code than the old pattern. --- src/application.rs | 10 +++++----- src/window.rs | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/application.rs b/src/application.rs index 0cf0142..1412699 100644 --- a/src/application.rs +++ b/src/application.rs @@ -99,11 +99,11 @@ glib::wrapper! { #[allow(clippy::new_without_default)] impl LpApplication { pub fn new() -> Self { - glib::Object::new(&[ - ("application-id", &config::APP_ID.to_string()), - ("flags", &gio::ApplicationFlags::HANDLES_OPEN), - ("resource-base-path", &"/org/gnome/Loupe/".to_string()), - ]) + glib::Object::builder() + .property("application-id", config::APP_ID) + .property("flags", gio::ApplicationFlags::HANDLES_OPEN) + .property("resource-base-path", "/org/gnome/Loupe") + .build() } pub fn setup_actions(&self) { diff --git a/src/window.rs b/src/window.rs index 9a6fe66..a28be8c 100644 --- a/src/window.rs +++ b/src/window.rs @@ -255,7 +255,7 @@ glib::wrapper! { #[gtk::template_callbacks] impl LpWindow { pub fn new>(app: &A) -> Self { - glib::Object::new(&[("application", app)]) + glib::Object::builder().property("application", app).build() } fn toggle_fullscreen(&self, fullscreen: bool) {