hiredis-client 0.7.4 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6b79457917d3d6fc74c57b37f880166344f4b7a35c911088a9044d67c42af1dc
-  data.tar.gz: 6b0b1c7e86c7b6a5ca1f4f09268455f5e4a5b8e6126a1c28d3552006d8adec52
+  metadata.gz: f18d435747c2eee26d1df005dc5f2dfe4941cec30a74baf90dd56d54a410c216
+  data.tar.gz: d38390db3e0a413525bfced8e86761e633cf5af0a7e7e8d28a90ed79e3930fac
 SHA512:
-  metadata.gz: 19e50e5a5e280cdf4b9246b69c7cb2f8b37cfe49c41bc86c1fea893a6fed6744173290a11d78e41a4fab8b71102ca268e62246b64bb244bdc6c4f2bd29d63415
-  data.tar.gz: b6e0f40b0ac5da4182dacf5894a2d6fa5d1ecae9f94e0fcf11c6e343f83bc9dd5cac91ddd89abd5c6710d56638d80ac1a91f527979c85d383dcf1992c623fe55
+  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.7.4
+  version: 0.8.1
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-09-06 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.7.4
+        version: 0.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.7.4
+        version: 0.8.1
 description:
 email:
 - jean.boussier@gmail.com