Updated run to return a Result rather than Never

.cargo/config.toml

@@ -5,6 +5,7 @@ rustflags = [
     "-Wclippy::pedantic",
     # globally allowed clippy exceptions
     "-Aclippy::module_name_repetitions",
+    "-Aclippy::missing_errors_doc",
     # Allow missing panic docs (see <https://github.com/rust-lang/rust-clippy/issues/11436>)
     "-Aclippy::missing_panics_doc",
 ]

crates/gooey-core/src/lib.rs

@@ -2,7 +2,8 @@
 
 use std::any::{type_name, Any};
 use std::collections::HashMap;
-use std::fmt::Debug;
+use std::error::Error;
+use std::fmt::{Debug, Display};
 use std::ops::{Deref, DerefMut};
 use std::panic::{RefUnwindSafe, UnwindSafe};
 use std::sync::Arc;
@@ -820,6 +821,31 @@ impl Debug for Child {
     }
 }
 
+pub struct EventLoopError(Box<dyn Error>);
+
+impl EventLoopError {
+    pub fn new<T>(error: T) -> Self
+    where
+        T: Error + 'static,
+    {
+        Self(Box::new(error))
+    }
+}
+
+impl Debug for EventLoopError {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        Debug::fmt(&self.0, f)
+    }
+}
+
+impl Display for EventLoopError {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        Display::fmt(&self.0, f)
+    }
+}
+
+impl Error for EventLoopError {}
+
 #[test]
 fn children_debug() {
     #[derive(Debug)]

crates/gooey-kludgine/src/lib.rs

@@ -7,7 +7,7 @@ use gooey_core::graphics::{Drawable, Options, TextMetrics};
 use gooey_core::math::units::UPx;
 use gooey_core::math::{IntoSigned, Point, Rect, ScreenUnit};
 use gooey_core::window::{NewWindow, Window, WindowAttributes, WindowLevel};
-use gooey_core::{Context, IntoNewWidget, NewWidget, Runtime, Widgets};
+use gooey_core::{Context, EventLoopError, IntoNewWidget, NewWidget, Runtime, Widgets};
 use gooey_raster::{DrawableState, RasterContext, Rasterizable, RasterizedApp, Surface};
 use kludgine::app::winit::dpi::{PhysicalPosition, PhysicalSize};
 use kludgine::app::winit::event::{ElementState, MouseButton};
@@ -17,11 +17,14 @@ use kludgine::shapes::Shape;
 use kludgine::text::TextOrigin;
 use kludgine::{Clipped, Color};
 
-pub fn run<Widget>(widgets: Arc<Widgets<RasterizedApp<Kludgine>>>, init: NewWindow<Widget>) -> !
+pub fn run<Widget>(
+    widgets: Arc<Widgets<RasterizedApp<Kludgine>>>,
+    init: NewWindow<Widget>,
+) -> Result<(), EventLoopError>
 where
     Widget: gooey_core::Widget,
 {
-    GooeyWindow::run_with((widgets, init))
+    GooeyWindow::run_with((widgets, init)).map_err(EventLoopError::new)
 }
 
 #[derive(Debug)]

examples/button.rs

@@ -1,9 +1,10 @@
 use std::string::ToString;
 
 use gooey::App;
+use gooey_core::EventLoopError;
 use gooey_widgets::Button;
 
-fn main() {
+fn main() -> Result<(), EventLoopError> {
     App::default().run(|cx, _window| {
         let counter = cx.new_dynamic(0i32);
         let label = counter.map_each(ToString::to_string).unwrap();

examples/counter.rs

@@ -1,10 +1,10 @@
 use std::string::ToString;
 
 use gooey::App;
-use gooey_core::Children;
+use gooey_core::{Children, EventLoopError};
 use gooey_widgets::{Button, Flex, Label};
 
-fn main() {
+fn main() -> Result<(), EventLoopError> {
     App::default().run(|cx, _window| {
         let counter = cx.new_dynamic(0i32);
         let label = counter.map_each(ToString::to_string).unwrap();

examples/counters.rs

@@ -1,8 +1,8 @@
 use gooey::App;
-use gooey_core::Children;
+use gooey_core::{Children, EventLoopError};
 use gooey_widgets::{Button, Flex};
 
-fn main() {
+fn main() -> Result<(), EventLoopError> {
     App::default().run(|cx, _window| {
         let counters = cx.new_dynamic(Children::new(cx));
 

examples/mouse_event.rs

@@ -1,6 +1,6 @@
 use gooey::App;
 use gooey_core::events::MouseEvent;
-use gooey_core::{Children, Context};
+use gooey_core::{Children, Context, EventLoopError};
 use gooey_widgets::{Button, Flex};
 
 fn position_button(cx: &Context) -> Button {
@@ -10,7 +10,7 @@ fn position_button(cx: &Context) -> Button {
     })
 }
 
-fn main() {
+fn main() -> Result<(), EventLoopError> {
     App::default().run(|cx, _window| {
         Flex::rows(Children::new(cx).with(position_button).with(|cx| {
             Flex::columns(

src/lib.rs

@@ -44,17 +44,22 @@ where
 
 #[cfg(all(feature = "web", target_arch = "wasm32"))]
 impl App<gooey_web::WebApp> {
-    pub fn run_with<Widget, Initializer>(self, init: Initializer) -> !
+    pub fn run_with<Widget, Initializer>(
+        self,
+        init: Initializer,
+    ) -> Result<(), gooey_core::EventLoopError>
     where
         Initializer: FnOnce(WindowBuilder) -> NewWindow<Widget>,
         Widget: gooey_core::Widget,
     {
         gooey_web::attach_to_body(self.widgets, init(WindowBuilder::default()));
-
-        wasm_bindgen::throw_str("This is not an actual error. Please ignore.");
+        Ok(())
     }
 
-    pub fn run<Widget, Initializer>(self, init: Initializer) -> !
+    pub fn run<Widget, Initializer>(
+        self,
+        init: Initializer,
+    ) -> Result<(), gooey_core::EventLoopError>
     where
         Initializer: FnOnce(&Context, &Window) -> Widget + std::panic::UnwindSafe + Send + 'static,
         Widget: gooey_core::Widget,
@@ -65,15 +70,21 @@ impl App<gooey_web::WebApp> {
 
 #[cfg(all(feature = "desktop", not(target_arch = "wasm32")))]
 impl App<gooey_raster::RasterizedApp<gooey_kludgine::Kludgine>> {
-    pub fn run_with<Widget, Initializer>(self, init: Initializer) -> !
+    pub fn run_with<Widget, Initializer>(
+        self,
+        init: Initializer,
+    ) -> Result<(), gooey_core::EventLoopError>
     where
         Initializer: FnOnce(WindowBuilder) -> NewWindow<Widget>,
         Widget: gooey_core::Widget,
     {
         gooey_kludgine::run(self.widgets, init(WindowBuilder::default()))
     }
 
-    pub fn run<Widget, Initializer>(self, init: Initializer) -> !
+    pub fn run<Widget, Initializer>(
+        self,
+        init: Initializer,
+    ) -> Result<(), gooey_core::EventLoopError>
     where
         Initializer: FnOnce(&Context, &Window) -> Widget + std::panic::UnwindSafe + Send + 'static,
         Widget: gooey_core::Widget,