hiredis-client 0.8.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 026fd9b702f8f490d0d2aa363e28ee61dabe8f0d477c8436e7df2460c356b0a4
-  data.tar.gz: 71cd542065a48fc85f3e90c9d19077048459575bfa72ab3987190d188cc33284
+  metadata.gz: 834e7347f7934e084b79647b8a179f2297e9e3ef29fce477a0d4a7c76ed99989
+  data.tar.gz: 121469ded1e076737990f7c6ce1ec86666cf4194dca2707fd1fa5645b63c3d6d
 SHA512:
-  metadata.gz: 1a9c6a8c8ddbfdc758ec14db5f42fb73dfc1bc3f1885478375f45017a22b191a7757aefffd22f0cc8db354b93513e69998b2c7b5b6d3266028d7ed629cb09064
-  data.tar.gz: 749f5192abd2c44d3f90a4c27db809f932377534f6bbc44256e1fb6948b112a2a61b9658a491e134d30d93798cdd43d378a40f69bf20bca44b6ef5b9b5bbc938
+  metadata.gz: 8f67fbd4582aaadbe6d7a2da0ce68bf45a9a712914e270de8e97511af579905033bfa5aa83c6055a38771078be87cdbb36e0883f5be471cfb3feb92cc5363007
+  data.tar.gz: 114644f547ddf1c707f76b04d53a8876d06f075aa66056d7185edcd86143643307dd06265e5f7a1446af0f207ac7c547f07e5ff9799c8b69b1816e4735c6f266

data/ext/redis_client/hiredis/hiredis_connection.c

@@ -48,6 +48,7 @@ static inline VALUE rb_hash_new_capa(long capa)
 
 static VALUE rb_eRedisClientCommandError, rb_eRedisClientConnectionError, rb_eRedisClientCannotConnectError, rb_eRedisClientProtocolError;
 static VALUE rb_eRedisClientReadTimeoutError, rb_eRedisClientWriteTimeoutError;
+static VALUE Redis_Qfalse;
 static ID id_parse;
 
 typedef struct {
@@ -200,8 +201,8 @@ static void *reply_create_nil(const redisReadTask *task) {
 static void *reply_create_bool(const redisReadTask *task, int bval) {
     reply_append(task, bval ? Qtrue : Qfalse);
     // Qfalse == NULL, so we can't return Qfalse as it would be interpreted as out of memory error.
-    // So we return Qnil instead.
-    return (void*)(bval ? Qtrue : Qnil);
+    // So we return a token value instead and the caller is responsible for turning it into Qfalse.
+    return (void*)(bval ? Qtrue : Redis_Qfalse);
 }
 
 static void reply_free(void *ptr) {
@@ -683,6 +684,10 @@ static VALUE hiredis_read(VALUE self) {
     if (hiredis_read_internal(connection, &reply)) {
         hiredis_raise_error_and_disconnect(connection, rb_eRedisClientReadTimeoutError);
     }
+    if (reply == Redis_Qfalse) {
+        // See reply_create_bool
+        reply = Qfalse;
+    }
     return reply;
 }
 
@@ -705,6 +710,8 @@ void Init_hiredis_connection(void) {
     redisInitOpenSSL();
 
     id_parse = rb_intern("parse");
+    Redis_Qfalse = rb_obj_alloc(rb_cObject);
+    rb_global_variable(&Redis_Qfalse);
 
     VALUE rb_cRedisClient = rb_const_get(rb_cObject, rb_intern("RedisClient"));
 

data/lib/redis_client/hiredis_connection.rb

@@ -29,6 +29,7 @@ class RedisClient
     end
 
     def initialize(config, connect_timeout:, read_timeout:, write_timeout:)
+      super()
       self.connect_timeout = connect_timeout
       self.read_timeout = read_timeout
       self.write_timeout = write_timeout

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hiredis-client
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.9.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-09-07 00:00:00.000000000 Z
+date: 2022-09-22 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.8.0
+        version: 0.9.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: 0.9.0
 description:
 email:
 - jean.boussier@gmail.com