protocol-zmtp 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0f82192f566af0256273b9dd794ad7032c5b1f3d1b2f8a58c37235847715f6fc
-  data.tar.gz: b74e78954d38be3f239d96cbcac676dc564b9ab56dff16f1beff8fbcedb77b8b
+  metadata.gz: a29e580def88791398cee267152d16bd7e187a518bd409f2e8a2acb1e364cab1
+  data.tar.gz: a27e2d1335fe8e0376abf7d17e3b9e4ed2d82e3c98d9dd70733327d789144576
 SHA512:
-  metadata.gz: 985cc52f11cc1ad7d24bdbec3916ee5485bda72a396413b3735254ae97430117b40ae3fd6331107e953c0b99bc3c05a997c368eeae642e64b023dd3d744381eb
-  data.tar.gz: 100df0f00b447bbc6799c4468b040f15e8fe633bf25d736b44931683772d5f52a50fa2646e8ce3f4768f6076d47482a9a004944f55f42a3d35720ca3f6ede27e
+  metadata.gz: 92318c07232337b3b108e5a98b4735c293e15810a8b41b7d1abe046f7f0ec7ed602072a91baa2ba6a6394ab05a4d260768e01a4302652ac96ca9ed61c9ad621d
+  data.tar.gz: 5e5fe073b34866af67c8b0e820afb95d37b2404cab2a986b26da24d70bb30fa809afef47a27c303b396f83a3f94c3f29da9e9a95f54f4413f83605b9d5fef8b1

data/lib/protocol/zmtp/codec/command.rb

@@ -23,11 +23,13 @@ module Protocol
         # @return [String] command data (binary)
         attr_reader :data
 
+        EMPTY_DATA = "".b.freeze
+
         # @param name [String] command name
         # @param data [String] command data
-        def initialize(name, data = "".b)
+        def initialize(name, data = EMPTY_DATA)
           @name = name
-          @data = data.b
+          @data = data.encoding == Encoding::BINARY ? data : data.b
         end
 
 
@@ -35,8 +37,9 @@ module Protocol
         #
         # @return [String] binary body (name-length + name + data)
         def to_body
-          name_bytes = @name.b
-          name_bytes.bytesize.chr.b + name_bytes + @data
+          name_bytes = @name.encoding == Encoding::BINARY ? @name : @name.b
+          buf = String.new(capacity: 1 + name_bytes.bytesize + @data.bytesize, encoding: Encoding::BINARY)
+          buf << Frame::FLAG_BYTES[name_bytes.bytesize] << name_bytes << @data
         end
 
 

data/lib/protocol/zmtp/codec/frame.rb

@@ -19,6 +19,12 @@ module Protocol
         # Short frame: 1-byte size, max body 255 bytes.
         SHORT_MAX = 255
 
+        # Pre-computed single-byte flag strings (avoids Integer#chr + String#b per frame).
+        FLAG_BYTES = Array.new(256) { |i| i.chr.b.freeze }.freeze
+
+        # Frozen empty binary string for zero-length frame bodies.
+        EMPTY_BODY = "".b.freeze
+
 
         # @return [String] frame body (binary)
         attr_reader :body
@@ -49,9 +55,9 @@ module Protocol
           flags |= FLAGS_COMMAND if @command
 
           if size > SHORT_MAX
-            (flags | FLAGS_LONG).chr.b + [size].pack("Q>") + @body
+            FLAG_BYTES[flags | FLAGS_LONG] + [size].pack("Q>") + @body
           else
-            flags.chr.b + size.chr.b + @body
+            FLAG_BYTES[flags] + FLAG_BYTES[size] + @body
           end
         end
 
@@ -64,9 +70,20 @@ module Protocol
         # @return [String] frozen binary wire representation
         #
         def self.encode_message(parts)
-          buf = +""
-          parts.each_with_index do |part, i|
-            buf << new(part, more: i < parts.size - 1).to_wire
+          buf  = String.new(encoding: Encoding::BINARY)
+          last = parts.size - 1
+          i    = 0
+          while i < parts.size
+            body  = parts[i]
+            body  = body.b unless body.encoding == Encoding::BINARY
+            size  = body.bytesize
+            flags = i < last ? FLAGS_MORE : 0
+            if size > SHORT_MAX
+              buf << FLAG_BYTES[flags | FLAGS_LONG] << [size].pack("Q>") << body
+            else
+              buf << FLAG_BYTES[flags] << FLAG_BYTES[size] << body
+            end
+            i += 1
           end
           buf.freeze
         end
@@ -95,7 +112,7 @@ module Protocol
             raise Error, "frame size #{size} exceeds max_message_size #{max_message_size}"
           end
 
-          body = size > 0 ? io.read_exactly(size) : "".b
+          body = size > 0 ? io.read_exactly(size) : EMPTY_BODY
 
           new(body, more: more, command: command)
         end

data/lib/protocol/zmtp/connection.rb

@@ -306,13 +306,17 @@ module Protocol
       def write_frames(parts)
         encrypted = @mechanism.encrypted?
         buf       = @header_buf
+        last      = parts.size - 1
 
-        parts.each_with_index do |part, i|
-          more = i < parts.size - 1
+        i = 0
+        while i < parts.size
+          part = parts[i]
+          more = i < last
           if encrypted
-            @io.write(@mechanism.encrypt(part.b, more: more))
+            body = part.encoding == Encoding::BINARY ? part : part.b
+            @io.write(@mechanism.encrypt(body, more: more))
           else
-            body  = part.b
+            body  = part.encoding == Encoding::BINARY ? part : part.b
             size  = body.bytesize
             flags = more ? Codec::Frame::FLAGS_MORE : 0
             buf.clear
@@ -324,6 +328,7 @@ module Protocol
             @io.write(buf)
             @io.write(body)
           end
+          i += 1
         end
       end
     end

data/lib/protocol/zmtp/version.rb

@@ -2,6 +2,6 @@
 
 module Protocol
   module ZMTP
-    VERSION = "0.5.0"
+    VERSION = "0.5.1"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: protocol-zmtp
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Patrik Wenger