Fix debugger and some tests

rune-rs · Feb 16, 2024 · 18c9a79 · 18c9a79
1 parent 5ee7e13
commit 18c9a79
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 17 deletions.
diff --git a/crates/rune/src/cli/run.rs b/crates/rune/src/cli/run.rs
@@ -273,9 +273,13 @@ pub(super) async fn run(
                 writeln!(io.stdout, "    *empty*")?;
             }
 
-            for (n, value) in stack.iter().enumerate() {
-                writeln!(io.stdout, "{}+{} = {:?}", frame.stack_bottom, n, value)?;
-            }
+            vm.with(|| {
+                for (n, value) in stack.iter().enumerate() {
+                    writeln!(io.stdout, "{}+{} = {:?}", frame.stack_bottom, n, value)?;
+                }
+
+                Ok::<_, crate::support::Error>(())
+            })?;
         }
 
         // NB: print final frame
@@ -292,15 +296,19 @@ pub(super) async fn run(
             writeln!(io.stdout, "    *empty*")?;
         }
 
-        for (n, value) in values.iter().enumerate() {
-            writeln!(
-                io.stdout,
-                "    {}+{} = {:?}",
-                stack.stack_bottom(),
-                n,
-                value
-            )?;
-        }
+        vm.with(|| {
+            for (n, value) in values.iter().enumerate() {
+                writeln!(
+                    io.stdout,
+                    "    {}+{} = {:?}",
+                    stack.stack_bottom(),
+                    n,
+                    value
+                )?;
+            }
+
+            Ok::<_, crate::support::Error>(())
+        })?;
     }
 
     if let Some(error) = errored {
@@ -402,9 +410,13 @@ where
                 writeln!(o, "    *empty*")?;
             }
 
-            for (n, value) in values.iter().enumerate() {
-                writeln!(o, "    {}+{} = {:?}", stack.stack_bottom(), n, value)?;
-            }
+            vm.with(|| {
+                for (n, value) in values.iter().enumerate() {
+                    writeln!(o, "    {}+{} = {:?}", stack.stack_bottom(), n, value)?;
+                }
+
+                Ok::<_, TraceError>(())
+            })?;
         }
 
         if let Some(result) = result {

diff --git a/crates/rune/src/modules/core.rs b/crates/rune/src/modules/core.rs
@@ -55,6 +55,7 @@ fn panic(message: &str) -> VmResult<()> {
 /// let value = Some(42);
 /// assert!(is_readable(value));
 /// let value2 = value.map(|v| v + 1);
+/// drop(value);
 /// assert!(!is_readable(value));
 /// assert_eq!(value2, Some(43));
 /// ```
@@ -74,6 +75,7 @@ fn is_readable(value: Value) -> bool {
 /// let value = Some(42);
 /// assert!(is_writable(value));
 /// let value2 = value.map(|v| v + 1);
+/// drop(value);
 /// assert!(!is_writable(value));
 /// assert_eq!(value2, Some(43));
 /// ```

diff --git a/crates/rune/src/modules/iter.rs b/crates/rune/src/modules/iter.rs
@@ -1,5 +1,7 @@
 //! The `std::iter` module.
 
+use core::convert::identity;
+
 use crate as rune;
 use crate::alloc::String;
 use crate::modules::collections::VecDeque;
@@ -43,7 +45,7 @@ pub fn module() -> Result<Module, ContextError> {
     module.function_meta(take)?;
     module.function_meta(count)?;
     module.associated_function(Protocol::NEXT, Iterator::next)?;
-    module.associated_function(Protocol::INTO_ITER, <Iterator as From<Iterator>>::from)?;
+    module.associated_function(Protocol::INTO_ITER, identity::<Iterator>)?;
 
     module.function_meta(range)?;
     module.function_meta(empty)?;

diff --git a/crates/rune/src/runtime/vm.rs b/crates/rune/src/runtime/vm.rs
@@ -2957,7 +2957,7 @@ impl Vm {
     /// vm.with(|| output.string_display(&mut f)).into_result()?;
     /// # Ok::<_, rune::support::Error>(())
     /// ```
-    pub fn with<F, T>(&mut self, f: F) -> T
+    pub fn with<F, T>(&self, f: F) -> T
     where
         F: FnOnce() -> T,
     {