Make Xilem Views must_use (#742)

Also fixes `unused_qualifications` as an easy fix.

---------

Co-authored-by: Kaur Kuut <[email protected]>

Author: DJMcNab

xilem/src/driver.rs

@@ -33,12 +33,10 @@ type MessagePackage = (Arc<[ViewId]>, DynMessage);
 
 impl RawProxy for MasonryProxy {
     fn send_message(&self, path: Arc<[ViewId]>, message: DynMessage) -> Result<(), ProxyError> {
-        match self
-            .0
-            .send_event(event_loop_runner::MasonryUserEvent::Action(
-                async_action(path, message),
-                ASYNC_MARKER_WIDGET,
-            )) {
+        match self.0.send_event(MasonryUserEvent::Action(
+            async_action(path, message),
+            ASYNC_MARKER_WIDGET,
+        )) {
             Ok(()) => Ok(()),
             Err(err) => {
                 let MasonryUserEvent::Action(masonry::Action::Other(res), _) = err.0 else {
@@ -74,7 +72,7 @@ where
     fn on_action(
         &mut self,
         masonry_ctx: &mut masonry::DriverCtx<'_>,
-        widget_id: masonry::WidgetId,
+        widget_id: WidgetId,
         action: masonry::Action,
     ) {
         let message_result = if widget_id == ASYNC_MARKER_WIDGET {

xilem/src/lib.rs

@@ -18,12 +18,13 @@
 )]
 // TODO: Remove any items listed as "Deferred"
 #![cfg_attr(not(debug_assertions), allow(unused))]
-#![expect(missing_debug_implementations, reason = "Deferred: Noisy")]
-#![expect(unused_qualifications, reason = "Deferred: Noisy")]
+#![expect(
+    missing_debug_implementations,
+    reason = "Deferred: Noisy. Requires same lint to be addressed in Masonry"
+)]
 #![expect(clippy::exhaustive_enums, reason = "Deferred: Noisy")]
 #![expect(clippy::match_same_arms, reason = "Deferred: Noisy")]
 #![expect(clippy::missing_assert_message, reason = "Deferred: Noisy")]
-#![expect(clippy::return_self_not_must_use, reason = "Deferred: Noisy")]
 #![expect(elided_lifetimes_in_paths, reason = "Deferred: Noisy")]
 #![expect(clippy::use_self, reason = "Deferred: Noisy")]
 // https://github.com/rust-lang/rust/pull/130025
@@ -65,6 +66,7 @@ pub mod view;
 pub use any_view::AnyWidgetView;
 pub use driver::{async_action, MasonryDriver, MasonryProxy, ASYNC_MARKER_WIDGET};
 
+#[must_use = "A Xilem app does nothing unless ran."]
 pub struct Xilem<State, Logic> {
     state: State,
     logic: Logic,

xilem/src/view/button.rs

@@ -35,6 +35,7 @@ pub fn button_any_pointer<State, Action>(
     }
 }
 
+#[must_use = "View values do nothing unless provided to Xilem."]
 pub struct Button<F> {
     label: ArcStr,
     callback: F,

xilem/src/view/checkbox.rs

@@ -22,6 +22,7 @@ where
     }
 }
 
+#[must_use = "View values do nothing unless provided to Xilem."]
 pub struct Checkbox<F> {
     label: ArcStr,
     checked: bool,
@@ -38,10 +39,7 @@ where
 
     fn build(&self, ctx: &mut ViewCtx) -> (Self::Element, Self::ViewState) {
         ctx.with_leaf_action_widget(|ctx| {
-            ctx.new_pod(masonry::widget::Checkbox::new(
-                self.checked,
-                self.label.clone(),
-            ))
+            ctx.new_pod(widget::Checkbox::new(self.checked, self.label.clone()))
         })
     }
 

xilem/src/view/flex.rs

@@ -27,6 +27,7 @@ pub fn flex<State, Action, Seq: FlexSequence<State, Action>>(
     }
 }
 
+#[must_use = "View values do nothing unless provided to Xilem."]
 pub struct Flex<Seq, State, Action = ()> {
     sequence: Seq,
     axis: Axis,

xilem/src/view/grid.rs

@@ -26,6 +26,7 @@ pub fn grid<State, Action, Seq: GridSequence<State, Action>>(
     }
 }
 
+#[must_use = "View values do nothing unless provided to Xilem."]
 pub struct Grid<Seq, State, Action = ()> {
     sequence: Seq,
     spacing: f64,

xilem/src/view/image.rs

@@ -31,6 +31,7 @@ pub fn image(image: &vello::peniko::Image) -> Image {
 /// The [`View`] created by [`image`].
 ///
 /// See `image`'s docs for more details.
+#[must_use = "View values do nothing unless provided to Xilem."]
 pub struct Image {
     image: vello::peniko::Image,
     object_fit: ObjectFit,

xilem/src/view/label.rs

@@ -17,6 +17,7 @@ pub fn label(label: impl Into<ArcStr>) -> Label {
     }
 }
 
+#[must_use = "View values do nothing unless provided to Xilem."]
 pub struct Label {
     label: ArcStr,
 
@@ -99,7 +100,7 @@ impl<State, Action> View<State, Action, ViewCtx> for Label {
         _id_path: &[ViewId],
         message: DynMessage,
         _app_state: &mut State,
-    ) -> crate::MessageResult<Action> {
+    ) -> MessageResult<Action> {
         tracing::error!("Message arrived in Label::message, but Label doesn't consume any messages, this is a bug");
         MessageResult::Stale(message)
     }

xilem/src/view/portal.rs

@@ -21,6 +21,7 @@ where
     }
 }
 
+#[must_use = "View values do nothing unless provided to Xilem."]
 pub struct Portal<V, State, Action> {
     child: V,
     phantom: PhantomData<(State, Action)>,

xilem/src/view/progress_bar.rs

@@ -20,9 +20,7 @@ impl<State, Action> View<State, Action, ViewCtx> for ProgressBar {
     type ViewState = ();
 
     fn build(&self, ctx: &mut ViewCtx) -> (Self::Element, Self::ViewState) {
-        ctx.with_leaf_action_widget(|ctx| {
-            ctx.new_pod(masonry::widget::ProgressBar::new(self.progress))
-        })
+        ctx.with_leaf_action_widget(|ctx| ctx.new_pod(widget::ProgressBar::new(self.progress)))
     }
 
     fn rebuild(