Cargo.lock

@@ -190,9 +190,6 @@ name = "anyhow"
 version = "1.0.86"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b3d1d046238990b9cf5bcde22a3fb3584ee5cf65fb2765f454ed428c7a0063da"
-dependencies = [
- "backtrace",
-]
 
 [[package]]
 name = "anymap"
@@ -781,7 +778,6 @@ checksum = "98cc8fbded0c607b7ba9dd60cd98df59af97e84d24e49c8557331cfc26d301ce"
 name = "cli"
 version = "0.1.0"
 dependencies = [
- "anyhow",
  "clap",
  "ctrlc",
  "env_logger",
@@ -1587,7 +1583,6 @@ dependencies = [
 name = "frontend-common"
 version = "0.5.1"
 dependencies = [
- "anyhow",
  "bytemuck",
  "chrono",
  "cpal",
@@ -1707,7 +1702,6 @@ dependencies = [
 name = "gb-emu-lib"
 version = "0.5.1"
 dependencies = [
- "anyhow",
  "ash 0.38.0+1.3.281",
  "ash-molten",
  "ash-window",
@@ -2213,7 +2207,6 @@ dependencies = [
 name = "gui"
 version = "0.1.0"
 dependencies = [
- "anyhow",
  "cacao",
  "cpal",
  "env_logger",
@@ -2644,6 +2637,7 @@ dependencies = [
 [[package]]
 name = "librashader"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "ash 0.37.3+1.3.251",
  "halfbrown",
@@ -2662,6 +2656,7 @@ dependencies = [
 [[package]]
 name = "librashader-cache"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "bincode",
  "blake3",
@@ -2677,6 +2672,7 @@ dependencies = [
 [[package]]
 name = "librashader-common"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "ash 0.37.3+1.3.251",
  "halfbrown",
@@ -2688,6 +2684,7 @@ dependencies = [
 [[package]]
 name = "librashader-preprocess"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "encoding_rs",
  "librashader-common",
@@ -2698,6 +2695,7 @@ dependencies = [
 [[package]]
 name = "librashader-presets"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "librashader-common",
  "nom",
@@ -2713,6 +2711,7 @@ dependencies = [
 [[package]]
 name = "librashader-reflect"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "bitflags 2.5.0",
  "bytemuck",
@@ -2734,6 +2733,7 @@ dependencies = [
 [[package]]
 name = "librashader-runtime"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "array-concat",
  "bytemuck",
@@ -2749,6 +2749,7 @@ dependencies = [
 [[package]]
 name = "librashader-runtime-vk"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "array-concat",
  "ash 0.37.3+1.3.251",
@@ -2768,6 +2769,7 @@ dependencies = [
 [[package]]
 name = "librashader-runtime-wgpu"
 version = "0.2.7"
+source = "git+https://git.alexjanka.com/alex/librashader#30bb747e94e7ab11ea5aecf388f4b693ac836434"
 dependencies = [
  "array-concat",
  "bytemuck",

cli/Cargo.toml

@@ -14,4 +14,3 @@ clap = { version = "4.4", features = ["derive"] }
 ctrlc = "3.4"
 log = { workspace = true }
 env_logger = { workspace = true }
-anyhow = { version = "1.0.86", features = ["backtrace"] }

cli/src/main.rs

@@ -121,7 +121,7 @@ impl From<Args> for frontend_common::RunOptions {
     }
 }
 
-fn main() -> anyhow::Result<()> {
+fn main() {
     env_logger::init();
     let args = Args::parse();
 
@@ -170,7 +170,7 @@ fn main() -> anyhow::Result<()> {
         let prepared = frontend_common::prepare(args.into(), receiver);
         let (output, stream) = audio::create_output(mute);
         let mut window_manager = ActiveWindowManager::new(sender, stream, record);
-        let mut core = frontend_common::run(prepared, &mut window_manager, output)?;
+        let mut core = frontend_common::run(prepared, &mut window_manager, output);
         if debug {
             let mut debugger = Debugger::new(Box::new(core));
             let mut since = Instant::now();
@@ -190,7 +190,6 @@ fn main() -> anyhow::Result<()> {
             window_manager.run_events_blocking().unwrap();
         }
     }
-    Ok(())
 }
 
 const UPDATE_INTERVAL: Duration = Duration::from_millis(1);

frontend-common/Cargo.toml

@@ -29,4 +29,3 @@ image = { version = "0.24", default-features = false, features = ["png"] }
 bytemuck = "1.14"
 chrono = "0.4"
 log = { workspace = true }
-anyhow = "1.0.86"

frontend-common/src/lib.rs

@@ -167,7 +167,7 @@ pub fn run<W>(
     prepared: PreparedEmulator,
     window_manager: &mut W,
     output: AudioOutput,
-) -> anyhow::Result<EmulatorCore<[u8; 4]>>
+) -> EmulatorCore<[u8; 4]>
 where
     W: WindowManager,
 {
@@ -178,16 +178,16 @@ where
         prepared.scale_override,
         prepared.shader_path,
         prepared.resizable,
-    )?;
+    );
 
     let tile_window = if prepared.tile_window {
-        Some(new_tile_window(window_manager)?)
+        Some(new_tile_window(window_manager))
     } else {
         None
     };
 
     let layer_window = if prepared.layer_window {
-        Some(new_layer_window(window_manager)?)
+        Some(new_layer_window(window_manager))
     } else {
         None
     };
@@ -224,7 +224,7 @@ where
     EmulatorCore::init(true, prepared.receiver, emulator_options)
 }
 
-pub fn new_tile_window<W>(window_manager: &mut W) -> anyhow::Result<RendererChannel>
+pub fn new_tile_window<W>(window_manager: &mut W) -> RendererChannel
 where
     W: WindowManager,
 {
@@ -236,7 +236,7 @@ where
     )
 }
 
-pub fn new_layer_window<W>(window_manager: &mut W) -> anyhow::Result<RendererChannel>
+pub fn new_layer_window<W>(window_manager: &mut W) -> RendererChannel
 where
     W: WindowManager,
 {

frontend-common/src/window.rs

@@ -21,5 +21,5 @@ pub trait WindowManager {
         factor: usize,
         shader_path: Option<PathBuf>,
         resizable: bool,
-    ) -> anyhow::Result<RendererChannel>;
+    ) -> RendererChannel;
 }

frontend-common/src/window/winit_manager.rs

@@ -56,7 +56,6 @@ where
 impl<Backend> WinitWindowManager<Backend>
 where
     Backend: RendererBackend,
-    <Backend as RendererBackend>::RendererError: Sync + Send + 'static,
 {
     pub fn new(
         sender: Sender<EmulatorMessage<[u8; 4]>>,
@@ -98,7 +97,6 @@ where
 impl<Backend> WindowManager for WinitWindowManager<Backend>
 where
     Backend: RendererBackend,
-    <Backend as RendererBackend>::RendererError: Sync + Send + 'static,
 {
     fn add(
         &mut self,
@@ -106,7 +104,7 @@ where
         factor: usize,
         shader_path: Option<PathBuf>,
         resizable: bool,
-    ) -> anyhow::Result<RendererChannel> {
+    ) -> RendererChannel {
         let is_main = window_type == WindowType::Main;
 
         self.data.add(
@@ -123,7 +121,6 @@ where
 impl<Backend> WinitWindowManagerData<Backend>
 where
     Backend: RendererBackend,
-    <Backend as RendererBackend>::RendererError: Sync + Send + 'static,
 {
     fn add(
         &mut self,
@@ -133,7 +130,7 @@ where
         event_loop: &EventLoop<()>,
         is_main: bool,
         record: bool,
-    ) -> anyhow::Result<RendererChannel> {
+    ) -> RendererChannel {
         let (r, sender) = WindowRenderer::new(
             factor,
             event_loop,
@@ -141,13 +138,13 @@ where
             shader_path,
             resizable,
             record,
-        )?;
+        );
         let id = r.window.id();
         self.windows.insert(id, r);
         if is_main {
             self.main_window = Some(id);
         }
-        Ok(sender)
+        sender
     }
 
     fn handler(&mut self, run_return: bool, event: Event<()>, target: &EventLoopWindowTarget<()>) {
@@ -323,7 +320,6 @@ impl QueuedBuf {
 impl<Backend> WindowRenderer<Backend>
 where
     Backend: RendererBackend,
-    <Backend as RendererBackend>::RendererError: Sync + Send + 'static,
 {
     #[allow(unused_variables)]
     fn new(
@@ -333,11 +329,12 @@ where
         shader_path: Option<PathBuf>,
         resizable: bool,
         record: bool,
-    ) -> anyhow::Result<(Self, RendererChannel)> {
+    ) -> (Self, RendererChannel) {
         let window = WindowBuilder::new()
             .with_title("Gameboy")
             .with_resizable(resizable)
-            .build(event_loop)?;
+            .build(event_loop)
+            .unwrap();
 
         let real_factor = (window.scale_factor() * factor as f64) as u32;
         let inner_size = window.inner_size();
@@ -347,9 +344,8 @@ where
             scaled_width: inner_size.width / real_factor,
             scaled_height: inner_size.height / real_factor,
         };
-        let renderer = RendererBackend::new(resolutions, &window, shader_path, manager);
 
-        let renderer = renderer?;
+        let renderer = RendererBackend::new(resolutions, &window, shader_path, manager).unwrap();
 
         let recording = if record {
             let configs = access_config();
@@ -369,7 +365,7 @@ where
 
         let (sender, receiver) = mpsc::channel();
 
-        Ok((
+        (
             Self {
                 receiver,
                 renderer,
@@ -381,7 +377,7 @@ where
                 recording,
             },
             sender,
-        ))
+        )
     }
 
     fn render(&mut self, manager: &Backend::RendererBackendManager) {

gui/Cargo.toml

@@ -31,7 +31,6 @@ log = { workspace = true }
 env_logger = { workspace = true }
 thiserror = { workspace = true }
 serde = { version = "1.0", features = ["derive"] }
-anyhow = "1.0.86"
 
 [target.'cfg(any(target_os = "macos"))'.dependencies]
 cacao = { git = "https://git.alexjanka.com/alex/cacao", optional = true }

gui/src/macos/cacao_window_manager.rs

@@ -335,7 +335,7 @@ impl WindowManager for CacaoWindowManager {
         factor: usize,
         shader_path: Option<std::path::PathBuf>,
         resizable: bool,
-    ) -> anyhow::Result<RendererChannel> {
+    ) -> RendererChannel {
         let (w, receiver) = CacaoWindow::new(
             window_type,
             factor,
@@ -364,7 +364,7 @@ impl WindowManager for CacaoWindowManager {
         window.show();
         self.windows
             .insert(window.delegate.as_ref().unwrap().window_type, window);
-        Ok(receiver)
+        receiver
     }
 }
 

lib/Cargo.toml

@@ -54,7 +54,6 @@ lazy_static = "1.4"
 wgpu = { version = "0.20", optional = true }
 thiserror = { workspace = true }
 log = { workspace = true }
-anyhow = "1.0.86"
 
 [build-dependencies]
 naga = { version = "0.19", optional = true, features = ["wgsl-in", "spv-out"] }

lib/src/connect/mod.rs

@@ -3,13 +3,12 @@ use std::path::PathBuf;
 use std::sync::mpsc::Sender;
 use std::sync::{Arc, RwLock};
 
-pub use crate::error::RomHeaderError;
 pub use crate::processor::memory::mmio::gpu::Colour;
 pub use crate::processor::memory::mmio::joypad::{JoypadButtons, JoypadState};
 pub use crate::processor::memory::mmio::serial::{SerialTarget, StdoutType};
 use crate::processor::memory::rom::sram_save::SaveDataLocation;
 pub use crate::processor::memory::rom::{
-    licensee::LicenseeCode, CartridgeType, CgbRomType, RamSize, RomHeader, RomSize,
+    licensee::LicenseeCode, CartridgeType, CgbRomType, RamSize, RomHeader, RomHeaderError, RomSize,
 };
 pub use crate::processor::memory::Rom;
 pub use crate::{HEIGHT, WIDTH};

lib/src/error.rs

@@ -1,55 +0,0 @@
-use thiserror::Error;
-
-#[derive(Debug, Error)]
-pub enum RomHeaderError {
-    #[error("slice not long enough for rom file")]
-    SliceLength,
-    #[error(transparent)]
-    Utf8(#[from] std::str::Utf8Error),
-    #[error("invalid ROM size")]
-    InvalidRomSize,
-    #[error("invalid RAM size")]
-    InvalidRamSize,
-    #[error("invalid MBC")]
-    InvalidMBC,
-}
-
-#[derive(Debug)]
-pub(crate) enum AddressError {
-    OutOfBounds,
-}
-
-#[cfg(feature = "pixels-renderer")]
-#[derive(Debug, Error)]
-pub enum PixelsError {
-    #[error("pixels error")]
-    Pixels(#[from] pixels::Error),
-}
-
-#[cfg(feature = "wgpu-renderer")]
-#[derive(Debug, Error)]
-pub enum WgpuError {
-    #[error("no adapter")]
-    NoAdapter,
-    #[error("no texture format")]
-    NoTextureFormat,
-    #[error("rwh error")]
-    RawWindowHandle(#[from] raw_window_handle::HandleError),
-    #[error("create surface error")]
-    CreateSurface(#[from] wgpu::CreateSurfaceError),
-    #[error("wgpu surface error")]
-    Surface(#[from] wgpu::SurfaceError),
-    #[error("request device error")]
-    RequestDevice(#[from] wgpu::RequestDeviceError),
-    #[error("librashader filterchain error")]
-    FilterChain(#[from] librashader::runtime::wgpu::error::FilterChainError),
-    #[error("couldn't load")]
-    CouldntLoad,
-}
-
-#[cfg(feature = "vulkan-renderer")]
-#[derive(Debug, Error)]
-pub enum VulkanError {
-    #[error("vulkan error")]
-    Vulkan, //(#[from] vk::Error),
-}

lib/src/lib.rs

@@ -1,7 +1,6 @@
 #![feature(let_chains, bigint_helper_methods)]
 
 use crate::processor::{memory::Memory, Flags};
-use anyhow::Context;
 use connect::{AudioOutput, EmulatorCoreTrait, EmulatorMessage, EmulatorOptions, RomFile};
 use processor::{
     memory::{mmio::gpu::Colour, rom::CgbRomType, OutputTargets},
@@ -9,7 +8,6 @@ use processor::{
 };
 use std::sync::mpsc::Receiver;
 
-pub mod error;
 pub mod renderer;
 
 #[cfg(feature = "config")]
@@ -33,13 +31,13 @@ where
 
 impl<ColourFormat> EmulatorCore<ColourFormat>
 where
-    ColourFormat: From<Colour> + Copy + Sync + Send + 'static,
+    ColourFormat: From<Colour> + Copy,
 {
     pub fn init(
         paused: bool,
         receiver: Receiver<EmulatorMessage<ColourFormat>>,
         options: EmulatorOptions<ColourFormat>,
-    ) -> anyhow::Result<Self> {
+    ) -> Self {
         let rom = options.rom;
 
         let is_cgb_mode = rom.rom_type == CgbRomType::CgbOnly || options.cgb_mode;
@@ -53,24 +51,28 @@ where
             ))
         }
         .load_data()
-        .context("couldn't load bootrom")?;
+        .expect("Error loading bootrom!");
 
         if let Some(window) = &options.window {
-            window.send(connect::RendererMessage::Prepare {
+            window
+                .send(connect::RendererMessage::Prepare {
                     width: WIDTH,
                     height: HEIGHT,
-            })?;
+                })
-            window.send(connect::RendererMessage::SetTitle {
+                .expect("message error");
+            window
+                .send(connect::RendererMessage::SetTitle {
                     title: format!(
                         "{} on {} on {}",
                         rom.get_title(),
                         rom.mbc_type(),
                         if is_cgb_mode { "CGB" } else { "DMG" }
                     ),
-            })?
+                })
+                .expect("message error");
         }
 
-        Ok(Self::new(
+        Self::new(
             paused,
             receiver,
             Cpu::new(
@@ -89,14 +91,9 @@ where
                 options.show_bootrom,
                 options.no_output,
             ),
-        ))
+        )
-    }
     }
 
-impl<ColourFormat> EmulatorCore<ColourFormat>
-where
-    ColourFormat: From<Colour> + Copy,
-{
     fn new(
         paused: bool,
         receiver: Receiver<EmulatorMessage<ColourFormat>>,

lib/src/processor/memory/addresses.rs

@@ -3,8 +3,6 @@ use std::{
     ops::{Add, Sub},
 };
 
-use crate::error::AddressError;
-
 pub(crate) use self::types::*;
 
 mod types;

lib/src/processor/memory/addresses/types.rs

@@ -3,7 +3,10 @@ use std::{
     ops::{Add, Sub},
 };
 
-use crate::error::AddressError;
+#[derive(Debug)]
+pub(crate) enum AddressError {
+    OutOfBounds,
+}
 
 #[derive(Copy, Clone, Debug)]
 pub(crate) struct BoundedAddress<const MIN: u16, const MAX: u16>(u16);

lib/src/processor/memory/rom.rs

@@ -1,5 +1,3 @@
-use crate::error::RomHeaderError;
-
 use self::{
     licensee::LicenseeCode,
     mbcs::{Mbc, Mbc1, Mbc2, Mbc3, Mbc5, None, KB, ROM_BANK_SIZE},
@@ -7,6 +5,7 @@ use self::{
 };
 use serde::{Deserialize, Serialize};
 use std::str::from_utf8;
+use thiserror::Error;
 
 use super::addresses::{CartRamAddress, RomAddress};
 
@@ -324,6 +323,20 @@ impl RomHeader {
     }
 }
 
+#[derive(Debug, Error)]
+pub enum RomHeaderError {
+    #[error("slice not long enough for rom file")]
+    SliceLength,
+    #[error("parsing UTF-8")]
+    Utf8(#[from] std::str::Utf8Error),
+    #[error("invalid ROM size")]
+    InvalidRomSize,
+    #[error("invalid RAM size")]
+    InvalidRamSize,
+    #[error("invalid MBC")]
+    InvalidMBC,
+}
+
 impl Rom {
     pub(crate) fn load(data: Vec<u8>, sram_location: Option<SaveDataLocation>) -> Self {
         let header_data = RomHeader::parse(&data).unwrap();

lib/src/renderer/pixels.rs

@@ -2,8 +2,9 @@ use std::{path::PathBuf, sync::Arc};
 
 use pixels::{Pixels, SurfaceTexture};
 use raw_window_handle::{DisplayHandle, HasDisplayHandle, HasWindowHandle};
+use thiserror::Error;
 
-use crate::{connect::ResolutionData, error::PixelsError};
+use crate::connect::ResolutionData;
 
 use super::{RendererBackend, RendererBackendManager};
 
@@ -81,6 +82,12 @@ fn new_pixels<W: HasDisplayHandle + HasWindowHandle>(
     .build()
 }
 
+#[derive(Debug, Error)]
+pub enum PixelsError {
+    #[error("pixels error")]
+    Pixels(#[from] pixels::Error),
+}
+
 struct DummyHandle<'a> {
     window: raw_window_handle::WindowHandle<'a>,
     display: raw_window_handle::DisplayHandle<'a>,

lib/src/renderer/vulkan/types.rs

@@ -13,6 +13,12 @@ use super::{
     VulkanBackendManager,
 };
 
+#[derive(Debug, Error)]
+pub enum VulkanError {
+    #[error("vulkan error")]
+    Vulkan, //(#[from] vk::Error),
+}
+
 pub(super) const SHADER_INPUT_FORMAT: vk::Format = vk::Format::R8G8B8A8_UNORM;
 
 #[derive(Clone, Debug, Copy)]

lib/src/renderer/wgpu.rs

@@ -3,11 +3,11 @@ use std::sync::Arc;
 use futures::executor::block_on;
 use librashader::runtime::wgpu::{FilterChain, FilterChainOptions};
 use raw_window_handle::{DisplayHandle, HasDisplayHandle, HasWindowHandle};
+use thiserror::Error;
 use wgpu::{Device, Queue, Surface, SurfaceConfiguration};
 
 use crate::{
     connect::ResolutionData,
-    error::WgpuError,
     renderer::{RendererBackend, RendererBackendManager},
 };
 
@@ -125,23 +125,23 @@ impl RendererBackend for WgpuBackend {
             let filter_chain_options = FilterChainOptions {
                 force_no_mipmaps: false,
             };
-            shader_path
+            match shader_path {
-                .and_then(|path| {
+                Some(path) => FilterChain::load_from_path(
-                    FilterChain::load_from_path(
                     path,
                     device.clone(),
                     queue.clone(),
                     Some(&filter_chain_options),
                 )
-                    .inspect_err(|e| log::error!("couldn't load shader: {e:?}"))
+                .unwrap(), //?,
-                    .ok()
+                None => {
-                })
+                    FilterChain::load_from_preset(
-                .unwrap_or(FilterChain::load_from_preset(
                         default_preset(),
                         device.clone(),
                         queue.clone(),
                         Some(&filter_chain_options),
-                )?)
+                    )
+                }?,
+            }
         };
 
         let gb_framebuffer = Arc::new(device.create_texture(&wgpu::TextureDescriptor {
@@ -268,3 +268,21 @@ impl RendererBackend for WgpuBackend {
         Ok(())
     }
 }
+
+#[derive(Debug, Error)]
+pub enum WgpuError {
+    #[error("no adapter")]
+    NoAdapter,
+    #[error("no texture format")]
+    NoTextureFormat,
+    #[error("rwh error")]
+    RawWindowHandle(#[from] raw_window_handle::HandleError),
+    #[error("create surface error")]
+    CreateSurface(#[from] wgpu::CreateSurfaceError),
+    #[error("wgpu surface error")]
+    Surface(#[from] wgpu::SurfaceError),
+    #[error("request device error")]
+    RequestDevice(#[from] wgpu::RequestDeviceError),
+    #[error("librashader filterchain error")]
+    FilterChain(#[from] librashader::runtime::wgpu::error::FilterChainError),
+}