summary refs log tree commit diff stats
path: root/tests/stdlib
diff options
context:
space:
mode:
authormetagn <metagngn@gmail.com>2024-09-04 10:25:01 +0300
committerGitHub <noreply@github.com>2024-09-04 09:25:01 +0200
commit080b0a03bd724ea50c5b90553c0aedc47be56a5e (patch)
tree1c5efc4db60fe9d566384d2bde45fbcbb50de81e /tests/stdlib
parentc314155fb37ef7dbd8a7de0df7e8b9d9b61f4802 (diff)
downloadNim-080b0a03bd724ea50c5b90553c0aedc47be56a5e.tar.gz
streams: implement readStr for VM, document VM limitations (#24058)
fixes #24054

`readData` is not implemented for the VM as mentioned in the issue, but
`readDataStr` is, so that is used for `readStr` instead on the VM. We
could also just use it in general since it falls back to `readData`
anyway but it's kept the same otherwise for now.

Also where and why streams in general don't work in VM is now documented
on the top level `streams` module documentation.
Diffstat (limited to 'tests/stdlib')
-rw-r--r--tests/stdlib/tstreams.nim4
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/stdlib/tstreams.nim b/tests/stdlib/tstreams.nim
index 0668d12bd..60c63b450 100644
--- a/tests/stdlib/tstreams.nim
+++ b/tests/stdlib/tstreams.nim
@@ -92,6 +92,10 @@ static: # Ensure streams it doesnt break with nimscript on arc/orc #19716
   let s = newStringStream("a")
   doAssert s.data == "a"
 
+static: # issue #24054, readStr
+  var s = newStringStream("foo bar baz")
+  doAssert s.readStr(3) == "foo"
+
 template main =
   var strm = newStringStream("abcde")
   var buffer = "12345"