hiredis-client 0.22.1 → 0.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36026e31c41cc2cfbc7e3e881f40e70b9aa41891c7c405b0966509b429b47b88
-  data.tar.gz: 44ae13fd2dd8bb04493e1f0140e27228fa2b6ecedf1e7c3738fa714ec2071bb2
+  metadata.gz: 44d5aa10a6591293ac181ec3fa1c332909e3c3ac9cf7ca6b02b6ec8679979f2e
+  data.tar.gz: 8105c8f1f67115e1f53425b0800281ea1cea1e402f212eee3ec71ae9b562b78d
 SHA512:
-  metadata.gz: 93ac4207ec72a7fc5353413cb4411dca05d542acf313aedbf32ee8033cb8a9b822e39973f023d52dadc481191ad4d06959a0e1b99806f33b9155bb9536304f0a
-  data.tar.gz: b3ff982553279b37562ff47fd43a0c2749e7587b209de8d3084a7a424037f76d8b5ede15545ae5000edc880d6f5d5003bdb755bafe107cb75c19eaf0e6fe2a23
+  metadata.gz: 2d972fc4d85e5af707f49c1811f3c7f5d5e971733790ca6d27f73ea81aa601f84193efb1a0f11dc375ad8b6bd336ef1455ec1469078579bcb1a7f8be84373cdf
+  data.tar.gz: 0f4e87b7fa8175cc7ee291bce633634bb69b6e23b67fa157cd3ad0a1509f44806a6e6c193fb77b1ee562d6bec172398a052f0a522de085222357af590af07d01

data/ext/redis_client/hiredis/hiredis_connection.c

@@ -719,7 +719,6 @@ static VALUE hiredis_flush(VALUE self) {
 
 static int hiredis_read_internal(hiredis_connection_t *connection, VALUE *reply) {
     void *redis_reply = NULL;
-    int wdone = 0;
 
     // This struct being on the stack, the GC won't move nor collect that `stack` RArray.
     // We use that to avoid having to have a `mark` function with write barriers.
@@ -737,56 +736,32 @@ static int hiredis_read_internal(hiredis_connection_t *connection, VALUE *reply)
         return HIREDIS_FATAL_CONNECTION_ERROR; // Protocol error
     }
 
-    if (redis_reply == NULL) {
-        /* Write until the write buffer is drained */
-        while (!wdone) {
-            errno = 0;
-
-            if (hiredis_buffer_write_nogvl(connection->context, &wdone) == REDIS_ERR) {
-                return HIREDIS_FATAL_CONNECTION_ERROR; // Socket error
-            }
-
-            if (errno == EAGAIN) {
-                int writable = 0;
-
-                if (hiredis_wait_writable(connection->context->fd, &connection->write_timeout, &writable) < 0) {
-                    return HIREDIS_CLIENT_TIMEOUT;
-                }
+    /* Read until there is a full reply */
+    while (redis_reply == NULL) {
+        errno = 0;
 
-                if (!writable) {
-                    errno = EAGAIN;
-                    return HIREDIS_CLIENT_TIMEOUT;
-                }
-            }
+        if (hiredis_buffer_read_nogvl(connection->context) == REDIS_ERR) {
+            return HIREDIS_FATAL_CONNECTION_ERROR; // Socket error
         }
 
-        /* Read until there is a full reply */
-        while (redis_reply == NULL) {
-            errno = 0;
+        if (errno == EAGAIN) {
+            int readable = 0;
 
-            if (hiredis_buffer_read_nogvl(connection->context) == REDIS_ERR) {
-                return HIREDIS_FATAL_CONNECTION_ERROR; // Socket error
+            if (hiredis_wait_readable(connection->context->fd, &connection->read_timeout, &readable) < 0) {
+                return HIREDIS_CLIENT_TIMEOUT;
             }
 
-            if (errno == EAGAIN) {
-                int readable = 0;
-
-                if (hiredis_wait_readable(connection->context->fd, &connection->read_timeout, &readable) < 0) {
-                    return HIREDIS_CLIENT_TIMEOUT;
-                }
-
-                if (!readable) {
-                    errno = EAGAIN;
-                    return HIREDIS_CLIENT_TIMEOUT;
-                }
-
-                /* Retry */
-                continue;
+            if (!readable) {
+                errno = EAGAIN;
+                return HIREDIS_CLIENT_TIMEOUT;
             }
 
-            if (redisGetReplyFromReader(connection->context, &redis_reply) == REDIS_ERR) {
-                return HIREDIS_FATAL_CONNECTION_ERROR; // Protocol error
-            }
+            /* Retry */
+            continue;
+        }
+
+        if (redisGetReplyFromReader(connection->context, &redis_reply) == REDIS_ERR) {
+            return HIREDIS_FATAL_CONNECTION_ERROR; // Protocol error
         }
     }
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hiredis-client
 version: !ruby/object:Gem::Version
-  version: 0.22.1
+  version: 0.23.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-04-16 00:00:00.000000000 Z
+date: 2024-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis-client
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.1
+        version: 0.23.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.1
+        version: 0.23.0
 description:
 email:
 - jean.boussier@gmail.com
@@ -79,7 +79,6 @@ files:
 - ext/redis_client/hiredis/vendor/test.c
 - ext/redis_client/hiredis/vendor/test.sh
 - ext/redis_client/hiredis/vendor/win32.h
-- hiredis-client.gemspec
 - lib/hiredis-client.rb
 - lib/redis_client/hiredis_connection.rb
 homepage: https://github.com/redis-rb/redis-client
@@ -105,7 +104,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.5
+rubygems_version: 3.0.3.1
 signing_key:
 specification_version: 4
 summary: Hiredis binding for redis-client

data/hiredis-client.gemspec

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../lib/redis_client/version"
-
-Gem::Specification.new do |spec|
-  spec.name = "hiredis-client"
-  spec.version = RedisClient::VERSION
-  spec.authors = ["Jean Boussier"]
-  spec.email = ["jean.boussier@gmail.com"]
-
-  spec.summary = "Hiredis binding for redis-client"
-  spec.homepage = "https://github.com/redis-rb/redis-client"
-  spec.license = "MIT"
-  spec.required_ruby_version = ">= 2.6.0"
-
-  spec.metadata["allowed_push_host"] = "https://rubygems.org"
-
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = spec.homepage
-  spec.metadata["changelog_uri"] = File.join(spec.homepage, "blob/master/CHANGELOG.md")
-
-  # Specify which files should be added to the gem when it is released.
-  # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files = Dir.chdir(File.expand_path(__dir__)) do
-    `git ls-files -z`.split("\x0").reject do |f|
-      (f == __FILE__) || f.match(%r{\A(?:(?:bin|test|spec|features|benchmark)/|\.(?:git|rubocop))})
-    end
-  end
-  spec.require_paths = ["lib"]
-  spec.extensions = ["ext/redis_client/hiredis/extconf.rb"]
-
-  spec.add_runtime_dependency "redis-client", RedisClient::VERSION
-end