impl inspect, ref

Author: uvlad7

README.md

@@ -62,6 +62,7 @@ You can also create a config and reuse it
 require "json_scanner"
 
 config = JsonScanner::Config.new([[], ["key"], [(0..-1)]])
+# => #<JsonScanner::Config [[], ['key'], [(0..9223372036854775807)]]>
 JsonScanner.scan('{"key": "42"}', config)
 # => [[[0, 13, :object]], [[8, 12, :string]], []]
 JsonScanner.scan('{"key": "42"}', config, with_path: true)

bin/console

@@ -1,6 +1,19 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
+#  Disable 'extensions are not built' warning
+module AnnoyingGemWarningSilencer
+  def contains_requirable_file?(file)
+    return super unless source.is_a?(Bundler::Source::Gemspec)
+    return false if respond_to?(:ignored?) && ignored?
+    return false if missing_extensions?
+
+    super
+  end
+end
+
+Gem::BasicSpecification.prepend(AnnoyingGemWarningSilencer)
+
 require "bundler/setup"
 require "json_scanner"
 

ext/json_scanner/json_scanner.c

@@ -651,6 +651,43 @@ VALUE config_m_initialize(VALUE self, VALUE path_ary)
   return self;
 }
 
+VALUE config_m_inspect(VALUE self)
+{
+  scan_ctx *ctx;
+  VALUE res;
+  TypedData_Get_Struct(self, scan_ctx, &config_type, ctx);
+  res = rb_sprintf("#<%" PRIsVALUE " [", rb_class_name(CLASS_OF(self)));
+  for (int i = 0; ctx->paths && i < ctx->paths_len; i++)
+  {
+    rb_str_cat_cstr(res, "[");
+    for (int j = 0; j < ctx->paths[i].len; j++)
+    {
+      switch (ctx->paths[i].elems[j].type)
+      {
+      case MATCHER_KEY:
+        rb_str_catf(res, "'%.*s'", (int)ctx->paths[i].elems[j].value.key.len, ctx->paths[i].elems[j].value.key.val);
+        break;
+      case MATCHER_INDEX:
+        rb_str_catf(res, "%ld", ctx->paths[i].elems[j].value.index);
+        break;
+      case MATCHER_INDEX_RANGE:
+        rb_str_catf(res, "(%ld..%ld)", ctx->paths[i].elems[j].value.range.start, ctx->paths[i].elems[j].value.range.end);
+        break;
+      case MATCHER_ANY_KEY:
+        rb_str_cat_cstr(res, "('*'..'*')");
+        break;
+      }
+      if (j < ctx->paths[i].len - 1)
+        rb_str_cat_cstr(res, ", ");
+    }
+    rb_str_cat_cstr(res, "]");
+    if (i < ctx->paths_len - 1)
+      rb_str_cat_cstr(res, ", ");
+  }
+  rb_str_cat_cstr(res, "]>");
+  return res;
+}
+
 static yajl_callbacks scan_callbacks = {
     scan_on_null,
     scan_on_boolean,
@@ -785,6 +822,7 @@ Init_json_scanner(void)
   rb_cJsonScannerConfig = rb_define_class_under(rb_mJsonScanner, "Config", rb_cObject);
   rb_define_alloc_func(rb_cJsonScannerConfig, config_alloc);
   rb_define_method(rb_cJsonScannerConfig, "initialize", config_m_initialize, 1);
+  rb_define_method(rb_cJsonScannerConfig, "inspect", config_m_inspect, 0);
   rb_define_const(rb_mJsonScanner, "ANY_INDEX", rb_range_new(INT2FIX(0), INT2FIX(-1), false));
   any_key_sym = rb_id2sym(rb_intern("*"));
   rb_define_const(rb_mJsonScanner, "ANY_KEY", rb_range_new(any_key_sym, any_key_sym, false));

spec/extensiontesttask.rb

@@ -51,15 +51,15 @@ def includes
         "../../#{@ext_dir}",
         "/usr/include/ruby-#{RUBY_VERSION}",
         "/usr/include/ruby-#{RUBY_VERSION}/#{RUBY_PLATFORM}",
-      ]).map { |l| "-I" + l }.join(" ")
+      ]).map { |l| "-I#{l}" }.join(" ")
     end
 
     def libraries
-      @libraries ||= (@test_libraries + %w[ruby pthread crypto]).map { |l| "-l" + l }.join(" ")
+      @libraries ||= (@test_libraries + %w[ruby pthread crypto]).map { |l| "-l#{l}" }.join(" ")
     end
 
     def lib_folders
-      @lib_folders ||= (@test_lib_folders + %w[/usr/lib .]).map { |l| "-L" + l }.join(" ")
+      @lib_folders ||= (@test_lib_folders + %w[/usr/lib .]).map { |l| "-L#{l}" }.join(" ")
     end
 
     def compile_tests

spec/json_scanner_spec.rb

@@ -234,10 +234,18 @@
       key = "abracadabra".dup
       conf = described_class.new [[], [key]]
       key["cad"] = 0.chr
-      key = nil
+      key = nil # rubocop:disable Lint/UselessAssignment
       GC.start
-      expect(10.times.map { JsonScanner.scan '{"abracadabra": 10}', conf, with_path: true }.uniq).to eq([[[[[], [0, 19, :object]]], [[["abracadabra"], [16, 18, :number]]]]])
-      expect(10.times.map { JsonScanner.scan '{"abracadabra": 10}', conf }.uniq).to eq([[[[0, 19, :object]], [[16, 18, :number]]]])
+      expect(
+        10.times.map do
+          JsonScanner.scan '{"abracadabra": 10}', conf, with_path: true
+        end.uniq,
+      ).to eq([[[[[], [0, 19, :object]]], [[["abracadabra"], [16, 18, :number]]]]])
+      expect(
+        10.times.map do
+          JsonScanner.scan '{"abracadabra": 10}', conf
+        end.uniq,
+      ).to eq([[[[0, 19, :object]], [[16, 18, :number]]]])
     end
 
     it "re-raises exceptions" do
@@ -251,5 +259,11 @@
         described_class.new [[(-42..1)]]
       end.to raise_error ArgumentError
     end
+
+    it "supports inspect" do
+      expect(
+        described_class.new([[], ["abracadabra", JsonScanner::ANY_INDEX], [42, JsonScanner::ANY_KEY]]).inspect,
+      ).to eq("#<JsonScanner::Config [[], ['abracadabra', (0..9223372036854775807)], [42, ('*'..'*')]]>")
+    end
   end
 end