numo-narray-alt 0.10.6 → 0.10.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad0c7d628c79e849dab290f3143d45bd4a1f755b1abd62851937f7ed7d894e93
-  data.tar.gz: 3643e1b645a95a1815237599543aa73735d0e19afa4ad7c10d4b95a902bd2df9
+  metadata.gz: 9897dbc5b94aa16af082880fe4085b341b628664ae6abc27e2ffa9e989e88451
+  data.tar.gz: d1d53a63934a8a51ee74cf64093b2b3869e03e75bea6070830230e14d9e91b91
 SHA512:
-  metadata.gz: e46752c80f12517f896253dbf2756b1b170c25032d4680fb71806f5157e00a565590118b31ea91208b10654fe68699acaf3e93b85adb174e35e03e83cadcc753
-  data.tar.gz: 646d182241adac8f628de8c18611cce09c44b1800737d9a1df10548ad99978ab96208f7efd5bb2ab57ba07f961785a1b42118ff7498b4a66f1fc6ea2838b5b8f
+  metadata.gz: 165941e9d06b5cd9e59ae591ab5666f9cb9924786d5253ce889da809de69b72c1893faef2a4c1e8f2d875021596415496802b8fb1bcf0b523b82c4cde231fb32
+  data.tar.gz: fd52d794a7c41344534a07e726b1f13bf0af8003d8e428a0c4f133a17efd5c9721e029a2365fc4a0f4cc04b21f9255897d4927f19500eeaeb51e2cdeb94018f1

data/ext/numo/narray/numo/narray.h

@@ -13,10 +13,10 @@ extern "C" {
 #endif
 #endif
 
-#define NARRAY_VERSION "0.10.6"
+#define NARRAY_VERSION "0.10.7"
 #define NARRAY_VERSION_MAJOR 0
 #define NARRAY_VERSION_MINOR 10
-#define NARRAY_VERSION_PATCH 6
+#define NARRAY_VERSION_PATCH 7
 #define NARRAY_VERSION_CODE                                                                    \
   (NARRAY_VERSION_MAJOR * 10000 + NARRAY_VERSION_MINOR * 100 + NARRAY_VERSION_PATCH)
 

data/ext/numo/narray/numo/types/robj_macro.h

@@ -68,13 +68,6 @@
 
 #define m_mulsum_init INT2FIX(0)
 
-#define m_sprintf(s, x) robj_sprintf(s, x)
-
-static inline int robj_sprintf(char* s, VALUE x) {
-  VALUE v = rb_funcall(x, rb_intern("to_s"), 0);
-  return sprintf(s, "%s", StringValuePtr(v));
-}
-
 #define m_sqrt(x)                                                                              \
   rb_funcall(rb_const_get(rb_mKernel, rb_intern("Math")), rb_intern("sqrt"), 1, x);
 

data/ext/numo/narray/src/mh/format.h

@@ -1,7 +1,7 @@
 #ifndef NUMO_NARRAY_MH_FORMAT_H
 #define NUMO_NARRAY_MH_FORMAT_H 1
 
-#define DEF_NARRAY_FORMAT_METHOD_FUNC(tDType)                                                  \
+#define DEF_NARRAY_FORMAT_ELEMENT_FUNC(tDType)                                                 \
   static VALUE format_##tDType(VALUE fmt, tDType* x) {                                         \
     if (NIL_P(fmt)) {                                                                          \
       char s[48];                                                                              \
@@ -9,8 +9,22 @@
       return rb_str_new(s, n);                                                                 \
     }                                                                                          \
     return rb_funcall(fmt, '%', 1, m_data_to_num(*x));                                         \
-  }                                                                                            \
-                                                                                               \
+  }
+
+/* robject formats its element without the fixed buffer above: the element is an
+   arbitrary Ruby object, so the length of its to_s is not bounded by anything
+   this side controls. */
+#define DEF_NARRAY_ROBJ_FORMAT_ELEMENT_FUNC()                                                  \
+  static VALUE format_robject(VALUE fmt, robject* x) {                                         \
+    if (NIL_P(fmt)) {                                                                          \
+      VALUE v = rb_funcall(*x, id_to_s, 0);                                                    \
+      StringValue(v);                                                                          \
+      return rb_str_new(RSTRING_PTR(v), RSTRING_LEN(v));                                       \
+    }                                                                                          \
+    return rb_funcall(fmt, '%', 1, m_data_to_num(*x));                                         \
+  }
+
+#define DEF_NARRAY_FORMAT_LOOP_METHOD_FUNC(tDType)                                             \
   static void iter_##tDType##_format(na_loop_t* const lp) {                                    \
     size_t n;                                                                                  \
     char* p1;                                                                                  \
@@ -52,4 +66,12 @@
     return na_ndloop(&ndf, 2, self, fmt);                                                      \
   }
 
+#define DEF_NARRAY_FORMAT_METHOD_FUNC(tDType)                                                  \
+  DEF_NARRAY_FORMAT_ELEMENT_FUNC(tDType)                                                       \
+  DEF_NARRAY_FORMAT_LOOP_METHOD_FUNC(tDType)
+
+#define DEF_NARRAY_ROBJ_FORMAT_METHOD_FUNC()                                                   \
+  DEF_NARRAY_ROBJ_FORMAT_ELEMENT_FUNC()                                                        \
+  DEF_NARRAY_FORMAT_LOOP_METHOD_FUNC(robject)
+
 #endif /* NUMO_NARRAY_MH_FORMAT_H */

data/ext/numo/narray/src/mh/sort.h

@@ -321,9 +321,9 @@ static inline void swapfunc(char* a, char* b, size_t n, int swaptype) {
     rb_funcall(idx, rb_intern("seq"), 0);                                                      \
     size_t size = na->size * sizeof(void*);                                                    \
     VALUE tmp;                                                                                 \
-    char* buf = rb_alloc_tmp_buffer(&tmp, size);                                               \
+    char* buf = RB_ALLOCV(tmp, size);                                                          \
     VALUE res = na_ndloop3(&ndf, buf, 3, self, idx, reduce);                                   \
-    rb_free_tmp_buffer(&tmp);                                                                  \
+    RB_ALLOCV_END(tmp);                                                                        \
     return res;                                                                                \
   }
 
@@ -475,9 +475,9 @@ static inline void swapfunc(char* a, char* b, size_t n, int swaptype) {
     rb_funcall(idx, rb_intern("seq"), 0);                                                      \
     size_t size = na->size * sizeof(void*);                                                    \
     VALUE tmp;                                                                                 \
-    char* buf = rb_alloc_tmp_buffer(&tmp, size);                                               \
+    char* buf = RB_ALLOCV(tmp, size);                                                          \
     VALUE res = na_ndloop3(&ndf, buf, 3, self, idx, reduce);                                   \
-    rb_free_tmp_buffer(&tmp);                                                                  \
+    RB_ALLOCV_END(tmp);                                                                        \
     return res;                                                                                \
   }
 

data/ext/numo/narray/src/t_robject.c

@@ -50,6 +50,7 @@ static ID id_reciprocal;
 static ID id_round;
 static ID id_square;
 static ID id_to_a;
+static ID id_to_s;
 static ID id_truncate;
 
 #include <numo/types/robject.h>
@@ -146,7 +147,7 @@ DEF_NARRAY_ASET_METHOD_FUNC(robject)
 DEF_NARRAY_COERCE_CAST_METHOD_FUNC(robject)
 DEF_NARRAY_TO_A_METHOD_FUNC(robject)
 DEF_NARRAY_FILL_METHOD_FUNC(robject)
-DEF_NARRAY_FORMAT_METHOD_FUNC(robject)
+DEF_NARRAY_ROBJ_FORMAT_METHOD_FUNC()
 DEF_NARRAY_FORMAT_TO_A_METHOD_FUNC(robject)
 DEF_NARRAY_ROBJ_INSPECT_METHOD_FUNC()
 DEF_NARRAY_EACH_METHOD_FUNC(robject)
@@ -370,6 +371,7 @@ void Init_numo_robject(void) {
   id_round = rb_intern("round");
   id_square = rb_intern("square");
   id_to_a = rb_intern("to_a");
+  id_to_s = rb_intern("to_s");
   id_truncate = rb_intern("truncate");
 
   /**

data/lib/numo/narray/extra.rb

@@ -137,6 +137,10 @@ module Numo
     #   # [[2, -3, 5],
     #   #  [4, 9, 7],
     #   #  [2, -1, 6]]
+    # @example
+    #   a = Numo::NArray.parse('true false nil')
+    #   # => Numo::Bit#shape=[1,3]
+    #   # [[1, 0, 0]]
 
     def self.parse(str, split1d: /\s+/, split2d: /;?$|;/,
                    split3d: /\s*\n(\s*\n)+/m)
@@ -151,7 +155,8 @@ module Numo
 
           c = []
           line.split(split1d).each do |item|
-            c << eval(item.strip) unless item.empty? # rubocop:disable Security/Eval
+            item = item.strip
+            c << parse_token(item) unless item.empty?
           end
           b << c unless c.empty?
         end
@@ -164,6 +169,42 @@ module Numo
       end
     end
 
+    # Convert one token of {parse} input into a value.
+    #
+    # Only the literals that describe an element are accepted: integer, float
+    # and complex numbers, and +true+, +false+ and +nil+, which {cast} turns
+    # into a {Numo::Bit}. Anything else -- an expression, a method call, a bare
+    # word -- raises ArgumentError, so text handed to {parse} is never executed
+    # as Ruby code.
+    #
+    # @param item [String] one whitespace-delimited token, already stripped
+    # @return [Integer, Float, Complex, true, false, nil] the parsed value
+    # @raise [ArgumentError] if the token is not one of those literals
+    def self.parse_token(item)
+      case item
+      when 'true' then return true
+      when 'false' then return false
+      when 'nil' then return nil
+      end
+
+      v = Integer(item, exception: false)
+      return v if v
+
+      v = Float(item, exception: false)
+      return v if v
+
+      if item.end_with?('i')
+        begin
+          return Complex(item)
+        rescue ArgumentError, TypeError
+          nil
+        end
+      end
+
+      raise ArgumentError, "invalid literal: #{item.inspect}"
+    end
+    private_class_method :parse_token
+
     # Iterate over an axis
     # @example
     #   > a = Numo::DFloat.new(2,2,2).seq

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: numo-narray-alt
 version: !ruby/object:Gem::Version
-  version: 0.10.6
+  version: 0.10.7
 platform: ruby
 authors:
 - yoshoku
@@ -214,7 +214,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/yoshoku/numo-narray-alt
   changelog_uri: https://github.com/yoshoku/numo-narray-alt/blob/main/CHANGELOG.md
-  documentation_uri: https://gemdocs.org/gems/numo-narray-alt/0.10.6/
+  documentation_uri: https://gemdocs.org/gems/numo-narray-alt/0.10.7/
   rubygems_mfa_required: 'true'
 post_install_message: |
   ===