listpack 0.0.1.beta3 → 0.0.1.beta4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf0000e6a9319e810ad90290e80bb6375bcaf8c91407f01d0caeb05bdca53ffd
-  data.tar.gz: d6e9fc6e06ac5ebbca6242e60bd5d1cca12a57b2bbcc6620d369179c1c32253f
+  metadata.gz: 4f59538906cf18c4c70e63a44cb070015653a8a3d0d7733707806fa83be30a01
+  data.tar.gz: 2400a85a4523e03df4ac447e768ae1c01a84b78fb00b14a28cec18a443d9cc54
 SHA512:
-  metadata.gz: 8c823632a71f7e2a334ac0cd2b8d437243103c31e034b3e3393c14f526d108fc7ada25fcc93bb9d6dc5447d7dd78073e204659e203839c81e095146fd1529361
-  data.tar.gz: 640eae14403530aaaf6ecf5a0de107edd839576a80be8f8aa5f2ae3f47faf431479cb4039776d669fc26ee949eeae22ab3f9673b390941ecef52c2ade68c4fcb
+  metadata.gz: 1a3430297a5b914fdcf4d174f7e541bdf681bca55ea92512f4e7416730ce6b3fd1be763219fc983c869bbe7d9e5b85dde5611d46e141b2ba4dad586b534cb539
+  data.tar.gz: 72b24a42842d6de03fde63303c5c1602d6693e59712646e1e708f961c62935255841d327fa3d621a775d38632a7cd505fd232e2eaa2d3711a429b9e0618f217f

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    listpack (0.0.1.beta3)
+    listpack (0.0.1.beta4)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -2,7 +2,6 @@
 
 Ruby wrapper for [Redis listpack](https://gist.github.com/antirez/66ffab20190ece8a7485bd9accfbc175) data structure
 
-
 ## Installation
 
 Add this line to your application's Gemfile:
@@ -22,29 +21,24 @@ Or install it yourself as:
 ## Usage
 
 ```ruby
-lp = Listpack.new
-
-lp.append('1234')
-lp.append('string')
+# Strings containing number are automatically converted to integers
+ary = ['1234', 'string']
+lp = Listpack.dump(ary)
+  # => "\x12\x00\x00\x00\x02\x00\xC4\xD2\x02\x86string\a\xFF"
 
-lp.to_s
+# This will be the same as the above
+ary = [1234, 'string']
+lp = Listpack.dump(ary)
   # => "\x12\x00\x00\x00\x02\x00\xC4\xD2\x02\x86string\a\xFF"
 ```
 
 #### Load a serialized listpack
 
 ```ruby
-data = "\x12\x00\x00\x00\x02\x00\xC4\xD2\x02\x86string\a\xFF"
-
-lp = Listpack.new(data)
-
-lp.size
-  # => 2
+lp = "\x12\x00\x00\x00\x02\x00\xC4\xD2\x02\x86string\a\xFF"
 
-lp.next
-  # => 1234
-lp.next
-  # => "string"
+ary = Listpack.load(lp)
+  # => [1234, "string"]
 ```
 
 ## Development

data/ext/listpack/listpack_rb.c

@@ -200,18 +200,28 @@ VALUE rb_class_listpack_dump(VALUE self, VALUE rb_ary)
   unsigned char *lp = lpNew();
   unsigned int i = 0;
   VALUE current;
+  VALUE retval;
 
   for (i = 0; i < len; i++) {
-      current = rb_ary_entry(rb_ary, i);
-      // TODO: convert integer to string if necessary
-      Check_Type(current, T_STRING);
-      lp = lpAppend(lp, (unsigned char *) RSTRING_PTR(current), RSTRING_LENINT(current));
-      if (lp == NULL) {
-        rb_raise(rb_eNoMemError, "no memory for listpack");
-      }
+    current = rb_ary_entry(rb_ary, i);
+
+    if (rb_type(current) == T_FIXNUM) {
+      current = rb_funcall(current, rb_intern("to_s"), 0);
+    } else if (rb_type(current) != T_STRING) {
+      lpFree(lp);
+      rb_raise(rb_eArgError, "wrong type");
+    }
+
+    lp = lpAppend(lp, (unsigned char *) RSTRING_PTR(current), RSTRING_LENINT(current));
+
+    if (lp == NULL) {
+      rb_raise(rb_eNoMemError, "no memory for listpack");
+    }
   }
 
-  return rb_str_new((char *) lp, lpBytes(lp));
+  retval = rb_str_new((char *) lp, lpBytes(lp));
+  lpFree(lp);
+  return retval;
 }
 
 VALUE rb_class_listpack_load(VALUE self, VALUE rb_str)

data/lib/listpack/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class Listpack
-  VERSION = '0.0.1.beta3'
+  VERSION = '0.0.1.beta4'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: listpack
 version: !ruby/object:Gem::Version
-  version: 0.0.1.beta3
+  version: 0.0.1.beta4
 platform: ruby
 authors:
 - Sebastian Wallin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-05-05 00:00:00.000000000 Z
+date: 2018-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler