brianmario-yajl-ruby 0.3.3 → 0.3.4

data/CHANGELOG.rdoc

@@ -1,5 +1,12 @@
 = Changelog
 
+0.3.4 (April 24th, 2009)
+ * turned Unicode checks back on in the Yajl parser now that it's fixed (thanks Lloyd!)
+ ** this also bumps the yajl version dependency requirement to 1.0.4
+ * better guessing of Integer/Float from number found instead of just trying to create a BigNum no matter what
+ * changed extconf.rb to fail Makefile creation if yajl isn't found
+ * added a test to check for parsing Infinity due to a Float overflow
+
 0.3.3 (April 24th, 2009)
  * 1.9 compatibility
 
@@ -16,13 +23,12 @@
  * added some initial spec tests
  ** ported some from ActiveSupport to ensure proper compatibility
  ** included 57 JSON fixtures to test against, all of which pass
- * changed parser config to not check for invalid unicode characters as Ruby is
- going to do this anyway (?). This resolves the remaining test failures around unicode.
+ * changed parser config to not check for invalid unicode characters as Ruby is going to do this anyway (?).
+ This resolves the remaining test failures around unicode.
  * changed how the parser was dealing with numbers to prevent overflows
- * added an exception class Yajl::ParseError which is now used in place of simply
- printing to STDERR upon a parsing error
- * renamed a couple of JSON test files in the benchmark folder to better represent their
- contents
+ * added an exception class Yajl::ParseError which is now used in place of simply printing to STDERR upon a parsing
+ error
+ * renamed a couple of JSON test files in the benchmark folder to better represent their contents
  * misc README updates
 
 0.2.1 (April 23rd, 2009)

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
-:patch: 3
+:patch: 4
 :major: 0
 :minor: 3

data/ext/extconf.rb

@@ -2,5 +2,9 @@ require 'mkmf'
 dir_config('yajl')
 have_header('yajl/yajl_parse.h')
 have_header('yajl/yajl_gen.h')
-have_library('yajl')
-create_makefile("yajl")
+
+if have_library("yajl")
+  create_makefile("yajl")
+else
+  puts "Yajl not found, maybe try manually specifying --with-yajl-dir to find it?"
+end

data/ext/yajl.c

@@ -48,7 +48,14 @@ static int found_boolean(void * ctx, int boolean) {
 }
 
 static int found_number(void * ctx, const char * numberVal, unsigned int numberLen) {
-    set_static_value(ctx, rb_str2inum(rb_str_new(numberVal, numberLen), 10));
+    if (strstr(numberVal, ".") != NULL
+        || strstr(numberVal, "e") != NULL
+        || strstr(numberVal, "E") != NULL) {
+        set_static_value(ctx, rb_Float(rb_str_new(numberVal, numberLen)));
+    } else {
+        set_static_value(ctx, rb_Integer(rb_str_new(numberVal, numberLen)));
+    }
+    
     return 1;
 }
 
@@ -101,7 +108,7 @@ static yajl_callbacks callbacks = {
 };
 
 static ID intern_io_read, intern_eof;
-yajl_parser_config cfg = {1, 0};
+static yajl_parser_config cfg = {1, 1};
 
 static VALUE t_parse(VALUE self, VALUE io) {
     yajl_handle hand;

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: brianmario-yajl-ruby
 version: !ruby/object:Gem::Version 
-  version: 0.3.3
+  version: 0.3.4
 platform: ruby
 authors: 
 - Brian Lopez