ucl 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 425cfe9bbc447dd450e023ce22cf7d87ff55999c44e9c662c478bfb04f27afdc
-  data.tar.gz: c71501063aa394c0f32f8a9e6ad66f4ca54a61309af6ee1f9e78d960d26711da
+  metadata.gz: 3fb8e8b518ea32adfa5365bf32e260487f3aa4ec9e546738911e6cf37a16e300
+  data.tar.gz: e9067e8747899c187f9ae400cd4bfe58e4343c41ff04b70cdbe161a500062bfc
 SHA512:
-  metadata.gz: dd42a5c178ca47bf3287b72bf7fa150ca21242c457cb8421590fa12f41a8d721079c3fe582335d85e7339c5cd5fd0b866dad2fa3a9569dba1eebf0460f4e2089
-  data.tar.gz: ca19582a1d3ef8882e46dae1da77c614d59cfe953b86fb05bc82e8dae491af6388acdf2085075c90b2f324870dc8ec26e6aebc8c8f6cd9d7f7513c334897b90a
+  metadata.gz: 962eea2f25228aec1ee903315074ceabd9cc424b2afab0963e68179fd39f1666abe6954404594ae6d4c658704b89200d4bfad21174bc9cd298b8ce66ea9252b7
+  data.tar.gz: b5c9bbe4b92be69734f5bd0df81a043dba97158489da7f67d7fce6e33f976a6eb209c45b7a72690bd441aa2fd8aed203f47389753328efd875fe05175e875afa

data/ext/ucl.c

@@ -25,6 +25,11 @@
 static VALUE mUCL                        = Qundef;
 static VALUE eUCLError                   = Qundef;
 
+static int ucl_allowed_flags = UCL_PARSER_KEY_LOWERCASE |
+	                       UCL_PARSER_NO_TIME       |
+	                       UCL_PARSER_DISABLE_MACRO |
+	                       UCL_PARSER_NO_FILEVARS   ;
+
 
 VALUE
 _iterate_valid_ucl(ucl_object_t const *root, int flags)
@@ -130,11 +135,6 @@ ucl_s_set_flags(VALUE klass, VALUE val)
 static VALUE
 ucl_s_parse(int argc, VALUE *argv, VALUE klass)
 {
-    static int allowed_flags = UCL_PARSER_KEY_LOWERCASE |
-	                       UCL_PARSER_NO_TIME       |
-	                       UCL_PARSER_DISABLE_MACRO |
-	                       UCL_PARSER_NO_FILEVARS   ;
-
     VALUE data, flags;
     rb_scan_args(argc, argv, "11", &data, &flags);
     if (NIL_P(flags)) flags = ucl_s_get_flags(mUCL);
@@ -142,7 +142,7 @@ ucl_s_parse(int argc, VALUE *argv, VALUE klass)
     rb_check_type(data,  T_STRING);
     rb_check_type(flags, T_FIXNUM);
 	
-    int c_flags = FIX2INT(flags) & allowed_flags;
+    int c_flags = FIX2INT(flags) & ucl_allowed_flags;
 
     struct ucl_parser *parser =
 	ucl_parser_new(c_flags | UCL_PARSER_NO_IMPLICIT_ARRAYS);
@@ -152,20 +152,16 @@ ucl_s_parse(int argc, VALUE *argv, VALUE klass)
 			 RSTRING_LEN(data));
     
     if (ucl_parser_get_error(parser)) {
-	rb_raise(eUCLError, "%s", ucl_parser_get_error(parser));
+	const char *errormsg = ucl_parser_get_error(parser);
+	if (parser != NULL) { ucl_parser_free(parser); }
+	rb_raise(eUCLError, "%s", errormsg);
     }
 
     ucl_object_t *root = ucl_parser_get_object(parser);
+    VALUE         res  = _iterate_valid_ucl(root, FIX2INT(flags));
 
-    VALUE res = _iterate_valid_ucl(root, FIX2INT(flags));
-
-    if (parser != NULL) {
-	ucl_parser_free(parser);
-    }
-    
-    if (root != NULL) {
-	ucl_object_unref(root);
-    }
+    if (parser != NULL) { ucl_parser_free(parser); }
+    if (root   != NULL) { ucl_object_unref(root);  }
     
     return res;
 }
@@ -187,17 +183,31 @@ ucl_s_load_file(int argc, VALUE *argv, VALUE klass)
 {
     VALUE file, flags;
     rb_scan_args(argc, argv, "11", &file, &flags);
+    if (NIL_P(flags)) flags = ucl_s_get_flags(mUCL);
 
-    VALUE read_kwargs = rb_hash_new();
-    rb_hash_aset(read_kwargs, rb_id2sym(rb_intern("encoding")),
-		 rb_str_new_cstr("BINARY"));
+    rb_check_type(file,  T_STRING);
+    rb_check_type(flags, T_FIXNUM);
+	
+    int   c_flags = FIX2INT(flags) & ucl_allowed_flags;
+    char *c_file  = StringValueCStr(file);
+    
+    struct ucl_parser *parser =
+	ucl_parser_new(c_flags | UCL_PARSER_NO_IMPLICIT_ARRAYS);
+    ucl_parser_add_file(parser, c_file);
+    
+    if (ucl_parser_get_error(parser)) {
+	const char *errormsg = ucl_parser_get_error(parser);
+	if (parser != NULL) { ucl_parser_free(parser); }
+	rb_raise(eUCLError, "%s", errormsg);
+    }
 
-    VALUE read_args[] = { file, read_kwargs };
-    VALUE data = rb_funcallv_kw(rb_cFile, rb_intern("read"),
-				2, read_args, RB_PASS_KEYWORDS);
+    ucl_object_t *root = ucl_parser_get_object(parser);
+    VALUE         res  = _iterate_valid_ucl(root, FIX2INT(flags));
 
-    VALUE parse_args[] = { data, flags };
-    return ucl_s_parse(2, parse_args, klass);
+    if (parser != NULL) { ucl_parser_free(parser); }
+    if (root   != NULL) { ucl_object_unref(root);  }
+    
+    return res;
 }
 
 

data/ucl.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
     s.name        = 'ucl'
-    s.version     = '0.1.1'
+    s.version     = '0.1.2'
     s.summary     = " Universal configuration library parser"
     s.description =  <<~EOF
       

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ucl
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Stéphane D'Alu