hiredis-client 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 026fd9b702f8f490d0d2aa363e28ee61dabe8f0d477c8436e7df2460c356b0a4
-  data.tar.gz: 71cd542065a48fc85f3e90c9d19077048459575bfa72ab3987190d188cc33284
+  metadata.gz: f18d435747c2eee26d1df005dc5f2dfe4941cec30a74baf90dd56d54a410c216
+  data.tar.gz: d38390db3e0a413525bfced8e86761e633cf5af0a7e7e8d28a90ed79e3930fac
 SHA512:
-  metadata.gz: 1a9c6a8c8ddbfdc758ec14db5f42fb73dfc1bc3f1885478375f45017a22b191a7757aefffd22f0cc8db354b93513e69998b2c7b5b6d3266028d7ed629cb09064
-  data.tar.gz: 749f5192abd2c44d3f90a4c27db809f932377534f6bbc44256e1fb6948b112a2a61b9658a491e134d30d93798cdd43d378a40f69bf20bca44b6ef5b9b5bbc938
+  metadata.gz: 1810b9bc04e01d77af516a659ac1dd565607b3473e19b3f88522b71609f5559fc0427d33e764fa4f147211c7e6e7440e2c8cc4e671d8bd6419d504398220709f
+  data.tar.gz: 96da9a05db116378d827bfbbedf11ff4e9cb634245fc5423c235f407934a95b54671e2cf90511fb528ef3af04d99d07c842add3deb8316a2bab3af3a66ed7fa1

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