binaryen 1.1.6.9 → 1.1.6.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18c535d2045e224d5ba46b3117e7eced2ff502b0cb338c6e7307113f011cb5fa
-  data.tar.gz: 8d1a5f09b94e86221e142d0773c8b23ea0a1987275a322a175daf84db2b2e01b
+  metadata.gz: 7a03d6e05a577d6579506e64e603e11f9430bf4d4f19c4d3f5cf0e984ad78387
+  data.tar.gz: c2995a3d92c07731d5b54b832c0d838159aebd2770dc1b4e0155bcc09ef2e7a8
 SHA512:
-  metadata.gz: dea3bf6b7f13d19fc83dc18632092306650ae3b077f2221fa62c0f35beaee50a4acf54ff50f114049da1bdf4b4485076f19e3039e9ec714b158468fb07286850
-  data.tar.gz: ac18a83b7221afaa90c2643ccad41af25f1be1bac141011afb056cc16a824e98c7c75832fe03b7f18b703184479faff1fe9bff127eb255cec321d50904a2e35e
+  metadata.gz: 01cdacd58083894fd4356f23032b00c00a4f586ac4d141742a50471228d772623027178e2b93879d9d224b3dc98fcdeb5c90374700914a7021752acbdaa78720
+  data.tar.gz: cfddf72d19ad3922e7152ec150d1ce31a936b54f25e1638674be47d453dd808353110ec481749977bd54a78369b2db3c7331342af7ae918a9cbef447261ec3f9

data/lib/binaryen/command.rb

@@ -7,11 +7,12 @@ require "tempfile"
 module Binaryen
   class Command
     include POSIX::Spawn
-    DEFAULT_MAX_OUTPUT_SIZE = 256 * 1024 * 1024
+    DEFAULT_MAX_OUTPUT_SIZE = 256 * 1024 * 1024 * 1024 # 256 MiB
     DEFAULT_TIMEOUT = 10
     DEFAULT_ARGS_FOR_COMMAND = {
       "wasm-opt" => ["--output=-"],
     }.freeze
+    BUFSIZE = 32 * 1024
 
     def initialize(cmd, timeout: DEFAULT_TIMEOUT, max_output_size: DEFAULT_MAX_OUTPUT_SIZE, ignore_missing: false)
       @cmd = command_path(cmd, ignore_missing) || raise(ArgumentError, "command not found: #{cmd}")
@@ -47,11 +48,9 @@ module Binaryen
 
     def spawn_command(*args, stderr: nil)
       out = "".b
-      data_buffer = "".b
       start_time = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       pid, stdin, stdout, stderr_stream = popen4(*args)
       stdin.close
-      @pid = pid
       readers = [stdout, stderr_stream]
 
       while readers.any?
@@ -61,33 +60,31 @@ module Binaryen
         raise Timeout::Error, "command timed out after #{@timeout} seconds" if ready.nil?
 
         ready[0].each do |io|
-          max_amount_to_read = @max_output_size - out.bytesize + 1
-          data = io.read_nonblock(max_amount_to_read, data_buffer, exception: false)
+          data = io.read_nonblock(BUFSIZE, exception: false)
 
           if data == :wait_readable
             # If the IO object is not ready for reading, read_nonblock returns :wait_readable.
             # This isn't an error, but a notification.
-            next
           elsif data.nil?
             # At EOF, read_nonblock returns nil instead of raising EOFError.
             readers.delete(io)
             io.close
           elsif io == stdout
-            out << data_buffer
+            out << data
           elsif io == stderr_stream && stderr
-            stderr << data_buffer
+            stderr << data
           end
         rescue Errno::EINTR
           # This means that the read was interrupted by a signal, which is not an error. So we just retry.
         end
 
         if out.bytesize > @max_output_size
-          Process.kill("TERM", @pid)
+          Process.kill("TERM", pid)
           raise Binaryen::MaximumOutputExceeded, "maximum output size exceeded (#{@max_output_size} bytes)"
         end
 
         if remaining_time < 0
-          Process.kill("TERM", @pid)
+          Process.kill("TERM", pid)
           raise Timeout::Error, "command timed out after #{@timeout} seconds"
         end
       end

data/lib/binaryen/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Binaryen
-  VERSION = "1.1.6.9"
+  VERSION = "1.1.6.10"
   BINARYEN_VERSION = "version_116"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: binaryen
 version: !ruby/object:Gem::Version
-  version: 1.1.6.9
+  version: 1.1.6.10
 platform: ruby
 authors:
 - Shopify Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-01-08 00:00:00.000000000 Z
+date: 2024-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: posix-spawn