uringmachine 0.28.1 → 0.28.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9fb662e62914ca38e254df5ee924b61155b510d3e862efa2f2e0997b00e7b3fd
-  data.tar.gz: 6ea55d8b64df3873677d1243342d5d6163812068392498f1f080351d0fc9d3e2
+  metadata.gz: 21a7c3c4503c06967ae77b6b3c8da3fb0f19eae3373efccb018ff111dc6d2aa8
+  data.tar.gz: 5376adb14245278692e634fdb6843080239a7a806b13032706b91036a9483c1a
 SHA512:
-  metadata.gz: 3ca70d442ad7b7379dccb7e1e6df6acac913f2b5ae9ac9a2ca87b8ab5ecac5efd00454888921faa7960321f91493ad10dd337ab043fc3b4771fdb6bc17dead0b
-  data.tar.gz: 6e4102474725dc14bf91690edcaf0e53d3eb9c7796b8edd72e2ec89629b7fc2c821e2a8c1e1abfaf52f0a27d9dd8153e6b750ed3f525436162bd93db7bf5aece
+  metadata.gz: 0b00614b471f51e8c3e31c77b2844284c371a4c0a73b30c0016969421ee20777ce2ca5ee215201a1a31aec189f7f703f39a0884c262c3ffc5c36ccbbae73a6f1
+  data.tar.gz: 26efdd3e98c013e9edd0962cc0fcab3bd9454485743fabc05a1d181200ad256aeb649618193048451850b39623f48cd69e9563f21defb8a178c5dcc300f24eb5

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 0.28.2 2026-02-20
+
+- Fix `Stream#get_string`
+
 # 0.28.1 2026-02-20
 
 - Add `Stream#machine`, `Stream#fd`

data/ext/um/um_stream.c

@@ -56,8 +56,8 @@ static inline void str_copy_bytes(VALUE dest, const char *src, ssize_t len) {
 }
 
 VALUE stream_get_line(struct um_stream *stream, VALUE buf, ssize_t maxlen) {
-  char *start = RSTRING_PTR(stream->buffer) + stream->pos;
   while (true) {
+    char *start = RSTRING_PTR(stream->buffer) + stream->pos;
     ssize_t pending_len = stream->len - stream->pos;
     ssize_t search_len = pending_len;
     ssize_t absmax_len = labs(maxlen);
@@ -76,32 +76,31 @@ VALUE stream_get_line(struct um_stream *stream, VALUE buf, ssize_t maxlen) {
       return buf;
     }
     else if (should_limit_len && pending_len > search_len)
-      // maxlen
+      // hit maxlen
       return Qnil;
 
     if (!stream_read_more(stream))
       return Qnil;
-    else
-      // update start ptr (it might have changed after reading)
-      start = RSTRING_PTR(stream->buffer) + stream->pos;
   }
 }
 
 VALUE stream_get_string(struct um_stream *stream, VALUE buf, ssize_t len) {
   size_t abslen = labs(len);
-  while (stream->len - stream->pos < abslen)
+  while (stream->len - stream->pos < abslen) {
     if (!stream_read_more(stream)) {
-      if (len > 0) return Qnil;
+      if (len > 0)
+        return Qnil;
 
       abslen = stream->len - stream->pos;
     }
+  }
 
   char *start = RSTRING_PTR(stream->buffer) + stream->pos;
   stream->pos += abslen;
 
   if (NIL_P(buf)) return rb_utf8_str_new(start, abslen);
 
-  str_copy_bytes(buf, start, len);
+  str_copy_bytes(buf, start, abslen);
   return buf;
 }
 

data/lib/uringmachine/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class UringMachine
-  VERSION = '0.28.1'
+  VERSION = '0.28.2'
 end

data/test/test_stream.rb

@@ -234,3 +234,20 @@ class StreamRespTest < StreamBaseTest
       s.resp_encode_cmd(+'', :set, 'foobar', :nx, :xx)
   end
 end
+
+class StreamHTTPTest < StreamBaseTest
+  def test_stream_http_etc
+    machine.write(@wfd, "GET / HTTP/1.1\r\n\r\nblahblah")
+    machine.close(@wfd)
+
+    l = @stream.get_line(nil, 0)
+    assert_equal "GET / HTTP/1.1", l
+
+    l = @stream.get_line(nil, 0)
+    assert_equal '', l
+
+    l = @stream.get_string(nil, -50)
+    assert_equal 'blahblah', l
+  end
+end
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: uringmachine
 version: !ruby/object:Gem::Version
-  version: 0.28.1
+  version: 0.28.2
 platform: ruby
 authors:
 - Sharon Rosner