Revert "Merge #48"

This reverts commit 8ab74cd, reversing
changes made to 442fb88.

Author: adamgreig

src/export.rs

@@ -8,44 +8,44 @@ use crate::hio::{self, HStderr, HStdout};
 
 static mut HSTDOUT: Option<HStdout> = None;
 
-pub fn hstdout_str(s: &str) {
-    let _result = interrupt::free(|_| unsafe {
+pub fn hstdout_str(s: &str) -> Result<(), ()> {
+    interrupt::free(|_| unsafe {
         if HSTDOUT.is_none() {
             HSTDOUT = Some(hio::hstdout()?);
         }
 
         HSTDOUT.as_mut().unwrap().write_str(s).map_err(drop)
-    });
+    })
 }
 
-pub fn hstdout_fmt(args: fmt::Arguments) {
-    let _result = interrupt::free(|_| unsafe {
+pub fn hstdout_fmt(args: fmt::Arguments) -> Result<(), ()> {
+    interrupt::free(|_| unsafe {
         if HSTDOUT.is_none() {
             HSTDOUT = Some(hio::hstdout()?);
         }
 
         HSTDOUT.as_mut().unwrap().write_fmt(args).map_err(drop)
-    });
+    })
 }
 
 static mut HSTDERR: Option<HStderr> = None;
 
-pub fn hstderr_str(s: &str) {
-    let _result = interrupt::free(|_| unsafe {
+pub fn hstderr_str(s: &str) -> Result<(), ()> {
+    interrupt::free(|_| unsafe {
         if HSTDERR.is_none() {
             HSTDERR = Some(hio::hstderr()?);
         }
 
         HSTDERR.as_mut().unwrap().write_str(s).map_err(drop)
-    });
+    })
 }
 
-pub fn hstderr_fmt(args: fmt::Arguments) {
-    let _result = interrupt::free(|_| unsafe {
+pub fn hstderr_fmt(args: fmt::Arguments) -> Result<(), ()> {
+    interrupt::free(|_| unsafe {
         if HSTDERR.is_none() {
             HSTDERR = Some(hio::hstderr()?);
         }
 
         HSTDERR.as_mut().unwrap().write_fmt(args).map_err(drop)
-    });
+    })
 }

src/macros.rs

@@ -20,18 +20,17 @@ macro_rules! syscall {
     };
 }
 
-/// Macro version of `syscall1`.
+/// Macro version of `syscall1`
 #[macro_export]
 macro_rules! syscall1 {
     ($nr:ident, $a1:expr) => {
         $crate::syscall1($crate::nr::$nr, $a1 as usize)
     };
 }
 
-/// Macro for printing to the HOST standard output.
+/// Macro for printing to the HOST standard output
 ///
-/// This is similar to the `print!` macro in the standard library. Both will panic on any failure to
-/// print.
+/// This macro returns a `Result<(), ()>` value
 #[macro_export]
 macro_rules! hprint {
     ($s:expr) => {
@@ -44,8 +43,7 @@ macro_rules! hprint {
 
 /// Macro for printing to the HOST standard output, with a newline.
 ///
-/// This is similar to the `println!` macro in the standard library. Both will panic on any failure to
-/// print.
+/// This macro returns a `Result<(), ()>` value
 #[macro_export]
 macro_rules! hprintln {
     () => {
@@ -59,10 +57,9 @@ macro_rules! hprintln {
     };
 }
 
-/// Macro for printing to the HOST standard error.
+/// Macro for printing to the HOST standard error
 ///
-/// This is similar to the `eprint!` macro in the standard library. Both will panic on any failure
-/// to print.
+/// This macro returns a `Result<(), ()>` value
 #[macro_export]
 macro_rules! heprint {
     ($s:expr) => {
@@ -75,8 +72,7 @@ macro_rules! heprint {
 
 /// Macro for printing to the HOST standard error, with a newline.
 ///
-/// This is similar to the `eprintln!` macro in the standard library. Both will panic on any failure
-/// to print.
+/// This macro returns a `Result<(), ()>` value
 #[macro_export]
 macro_rules! heprintln {
     () => {
@@ -90,23 +86,22 @@ macro_rules! heprintln {
     };
 }
 
-/// Macro that prints and returns the value of a given expression for quick and
-/// dirty debugging.
-///
-/// Works exactly like `dbg!` in the standard library, replacing `eprintln!`
-/// with `heprintln!`.
+/// Macro that prints and returns the value of a given expression
+/// for quick and dirty debugging. Works exactly like `dbg!` in
+/// the standard library, replacing `eprintln` with `heprintln`,
+/// which it unwraps.
 #[macro_export]
 macro_rules! dbg {
     () => {
-        $crate::heprintln!("[{}:{}]", file!(), line!());
+        $crate::heprintln!("[{}:{}]", file!(), line!()).unwrap();
     };
     ($val:expr) => {
         // Use of `match` here is intentional because it affects the lifetimes
         // of temporaries - https://stackoverflow.com/a/48732525/1063961
         match $val {
             tmp => {
                 $crate::heprintln!("[{}:{}] {} = {:#?}",
-                    file!(), line!(), stringify!($val), &tmp);
+                    file!(), line!(), stringify!($val), &tmp).unwrap();
                 tmp
             }
         }