Remove direct eprintln! usage from vm

Author: youknowone

stdlib/src/faulthandler.rs

@@ -2,10 +2,14 @@ pub(crate) use decl::make_module;
 
 #[pymodule(name = "faulthandler")]
 mod decl {
-    use crate::vm::{frame::FrameRef, function::OptionalArg, VirtualMachine};
+    use crate::vm::{
+        frame::FrameRef, function::OptionalArg, stdlib::sys::PyStderr, VirtualMachine,
+    };
 
-    fn dump_frame(frame: &FrameRef) {
-        eprintln!(
+    fn dump_frame(frame: &FrameRef, vm: &VirtualMachine) {
+        let stderr = PyStderr(vm);
+        writeln!(
+            stderr,
             "  File \"{}\", line {} in {}",
             frame.code.source_path,
             frame.current_location().row(),
@@ -19,10 +23,11 @@ mod decl {
         _all_threads: OptionalArg<bool>,
         vm: &VirtualMachine,
     ) {
-        eprintln!("Stack (most recent call first):");
+        let stderr = PyStderr(vm);
+        writeln!(stderr, "Stack (most recent call first):");
 
         for frame in vm.frames.borrow().iter() {
-            dump_frame(frame);
+            dump_frame(frame, vm);
         }
     }
 

vm/src/stdlib/warnings.rs

@@ -5,6 +5,7 @@ mod _warnings {
     use crate::{
         builtins::{PyStrRef, PyTypeRef},
         function::OptionalArg,
+        stdlib::sys::PyStderr,
         AsObject, PyResult, VirtualMachine,
     };
 
@@ -33,7 +34,14 @@ mod _warnings {
         } else {
             vm.ctx.exceptions.user_warning.clone()
         };
-        eprintln!("level:{}: {}: {}", level, category.name(), args.message);
+        let stderr = PyStderr(vm);
+        writeln!(
+            stderr,
+            "level:{}: {}: {}",
+            level,
+            category.name(),
+            args.message
+        );
         Ok(())
     }
 }