xorcist 1.0.1 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2de9dd2f8528b30b41521656e48ba9b4a1e5e39f
-  data.tar.gz: d791dcdd4766e8aef412047954e1504464ee6584
+  metadata.gz: e60583bfba07126835c5def71dbfd1997f504a22
+  data.tar.gz: 13cf6b74ab9ea7ae040148ce130e24665c73bd25
 SHA512:
-  metadata.gz: d9f69c450e68d80ca5cf76d32a72930ba8e3436ef1252ada6808eebc85273d4de805579a6f60a2c9e4924c514dd6ce653d5e9fc1c634f8c33c8a9e70f39ea611
-  data.tar.gz: 859066a86e1c7879f378d202adef1c179fba1f4d6f913ac4fcd161d575a4c2e86a2926745e0e8a13a0f153fba3d862a604cc03ea89742130c9c29b5cc4a4062c
+  metadata.gz: 391f4eae4f0422e52ad1e52ddd500f23c23373a4a1d2c8e5004c70dc3a4f21364e8d9e3fedfc4fabb2bf8a31e72129477fdca5efb0709f136005eecfcaaff99a
+  data.tar.gz: 7d884189cb4f58d98ce01a6dd72ff97d1e95507c292f3960f7b345220c3e87af040eb5cd57800cb147fe7122e6c636b01b6349843e43092ed7546962ff1260bb

data/README.md

@@ -15,6 +15,7 @@ Xorcist.xor(a, b) # => "\u0003"
 a # => 'a'
 Xorcist.xor!(a, b) # => "\u0003"
 a # => "\u0003"
+Xorcist.xor!(a.freeze, b) # => RuntimeError!
 ```
 
 You can `include Xorcist` to expose its methods:
@@ -70,7 +71,7 @@ Calculating -------------------------------------
 ### JRuby 1.7.19
 
 ```
-Picked up JAVA_TOOL_OPTIONS: -javaagent:/usr/share/java/jayatanaag.jar 
+Picked up JAVA_TOOL_OPTIONS: -javaagent:/usr/share/java/jayatanaag.jar
 Calculating -------------------------------------
                 ruby    31.795k i/100ms
              xorcist   118.084k i/100ms

data/ext/xorcist/extconf.rb

@@ -1,4 +1,4 @@
 require 'mkmf'
 extension_name = 'xorcist'
 dir_config(extension_name)
-create_makefile(extension_name)
+create_makefile('xorcist/xorcist')

data/ext/xorcist/xorcist.c

@@ -4,7 +4,7 @@ VALUE Xorcist = Qnil;
 
 void Init_xorcist();
 
-VALUE xor_in_place(VALUE x, VALUE y, VALUE self);
+VALUE xor_in_place(VALUE self, VALUE x, VALUE y);
 
 VALUE xor_in_place(VALUE self, VALUE x, VALUE y) {
     const char *src = 0;
@@ -12,6 +12,16 @@ VALUE xor_in_place(VALUE self, VALUE x, VALUE y) {
     size_t len;
     size_t y_len;
 
+    if (TYPE(x) != T_STRING) {
+        rb_raise( rb_eTypeError, "first argument must be a String" );
+        return Qnil;
+    }
+
+    if (TYPE(y) != T_STRING) {
+        rb_raise( rb_eTypeError, "second argument must be a String" );
+        return Qnil;
+    }
+
     rb_str_modify(x);
     dest = RSTRING_PTR(x);
     len = RSTRING_LEN(x);

data/lib/xorcist.rb

@@ -1,10 +1,10 @@
 require 'xorcist/version'
 
 if RUBY_ENGINE == 'jruby'
-  require 'jruby'
+  require 'java'
   require File.expand_path('../xorcist.jar', __FILE__)
 else
-  require File.expand_path('../xorcist.so', __FILE__)
+  require 'xorcist/xorcist'
 end
 
 module Xorcist
@@ -12,4 +12,18 @@ module Xorcist
   def xor(x, y)
     xor!(x.dup, y)
   end
+
+  # rb_str_modify isn't blowing up for frozen strings under Rubinius, so we'll
+  # hack in a fix here
+  if RUBY_ENGINE == 'rbx'
+    alias_method :_xor!, :xor!
+    remove_method :xor!
+    def xor!(x, y)
+      if x.frozen?
+        raise(RuntimeError, "can't modify frozen String")
+      else
+        _xor!(x, y)
+      end
+    end
+  end
 end

data/lib/xorcist/version.rb

@@ -1,3 +1,3 @@
 module Xorcist
-  VERSION = '1.0.1'
+  VERSION = '1.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: xorcist
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.1.0
 platform: ruby
 authors:
 - Faraz Yashar
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-25 00:00:00.000000000 Z
+date: 2018-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -102,7 +102,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.6
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Blazing-fast-cross-platform-monkey-patch-free string XOR