Data -> Config

Author: sue445

lib/ruby_header_parser.rb

@@ -10,7 +10,7 @@ module RubyHeaderParser
   class Error < StandardError; end
 
   autoload :ArgumentDefinition, "ruby_header_parser/argument_definition"
-  autoload :Data,               "ruby_header_parser/data"
+  autoload :Config,             "ruby_header_parser/config"
   autoload :EnumDefinition,     "ruby_header_parser/enum_definition"
   autoload :FunctionDefinition, "ruby_header_parser/function_definition"
   autoload :Parser,             "ruby_header_parser/parser"

lib/ruby_header_parser/data.rb renamed to lib/ruby_header_parser/config.rb

@@ -2,7 +2,7 @@
 
 module RubyHeaderParser
   # Manager for config file
-  class Data
+  class Config
     # @!attribute [r] data
     #   @return [Hash]
     attr_reader :data

lib/ruby_header_parser/parser.rb

@@ -16,8 +16,8 @@ class Parser # rubocop:disable Metrics/ClassLength
     attr_reader :dist_preprocessed_header_file
 
     # @!attribute [r] data
-    #   @return [RubyHeaderParser::Data]
-    attr_reader :data
+    #   @return [RubyHeaderParser::Config]
+    attr_reader :config
 
     DEFAULT_HEADER_FILE = "#{RbConfig::CONFIG["rubyhdrdir"]}/ruby.h".freeze
 
@@ -38,7 +38,7 @@ def initialize(dist_preprocessed_header_file: nil, header_file: DEFAULT_HEADER_F
       @header_file = header_file
       @include_paths = include_paths
       @dist_preprocessed_header_file = dist_preprocessed_header_file || File.join(Dir.tmpdir, "ruby_preprocessed.h")
-      @data = Data.new
+      @config = Config.new
     end
 
     # @return [Array<RubyHeaderParser::FunctionDefinition>]
@@ -59,7 +59,7 @@ def extract_struct_definitions
         parts = line.split("\t")
 
         struct_name = parts[0]
-        next unless data.should_generate_struct?(struct_name)
+        next unless config.should_generate_struct?(struct_name)
 
         definitions << StructDefinition.new(
           name: struct_name,
@@ -76,7 +76,7 @@ def extract_type_definitions
 
         type_name = parts[0]
 
-        next unless data.should_generate_type?(type_name)
+        next unless config.should_generate_type?(type_name)
 
         definitions << TypeDefinition.new(
           name: type_name,
@@ -97,7 +97,7 @@ def extract_enum_definitions
 
           value = parts[0]
 
-          next unless data.should_generate_enum?(enum_name)
+          next unless config.should_generate_enum?(enum_name)
 
           hash[enum_name] ||= EnumDefinition.new(name: enum_name)
           hash[enum_name].values << value
@@ -131,7 +131,7 @@ def generate_function_definition_from_line(line:, kind:, is_parse_multiline_defi
       function_name = parts[0]
       filepath = parts[1]
 
-      return nil unless data.should_generate_function?(function_name)
+      return nil unless config.should_generate_function?(function_name)
 
       return nil unless parts[3] == kind
 
@@ -228,7 +228,7 @@ def create_typeref(definition:, function_name:, typeref_field:, filepath:, line_
       typeref_pointer = nil
       if typeref_type.match?(/\*+$/)
         typeref_type = typeref_type.gsub(/\*+$/, "").strip
-        typeref_pointer = data.function_self_pointer_hint(function_name)
+        typeref_pointer = config.function_self_pointer_hint(function_name)
       end
 
       TyperefDefinition.new(type: typeref_type, pointer: typeref_pointer)
@@ -319,12 +319,12 @@ def analyze_argument_type(function_name:, arg_pos:, parts:)
       case original_type
       when /\*+$/
         type = original_type.gsub(/\*+$/, "").strip
-        pointer = data.function_arg_pointer_hint(function_name:, pos: arg_pos)
+        pointer = config.function_arg_pointer_hint(function_name:, pos: arg_pos)
 
       when /^void\s*/, /\(.*\)/
         # function pointer (e.g. void *(*func)(void *)) is treated as `void*`
         type = "void"
-        pointer = data.function_arg_pointer_hint(function_name:, pos: arg_pos)
+        pointer = config.function_arg_pointer_hint(function_name:, pos: arg_pos)
 
       else
         type = original_type

sig/ruby_header_parser/data.rbs renamed to sig/ruby_header_parser/config.rbs

@@ -1,5 +1,5 @@
 module RubyHeaderParser
-  class Data
+  class Config
     attr_reader data: Hash[untyped, untyped]
 
     def initialize: () -> void

sig/ruby_header_parser/parser.rbs

@@ -3,7 +3,7 @@ module RubyHeaderParser
     attr_reader header_file: String
     attr_reader include_paths: Array[String]
     attr_reader dist_preprocessed_header_file: String
-    attr_reader data: Data
+    attr_reader config: Config
 
     DEFAULT_HEADER_FILE: String
     DEFAULT_INCLUDE_PATHS: Array[String]

spec/ruby_header_parser/data_spec.rb renamed to spec/ruby_header_parser/config_spec.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-RSpec.describe RubyHeaderParser::Data do
-  let(:data) { RubyHeaderParser::Data.new }
+RSpec.describe RubyHeaderParser::Config do
+  let(:config) { RubyHeaderParser::Config.new }
 
   describe "#function_arg_pointer_hint" do
-    subject { data.function_arg_pointer_hint(function_name:, pos:) }
+    subject { config.function_arg_pointer_hint(function_name:, pos:) }
 
     context "found in config/default.yml" do
       let(:function_name) { "rb_funcallv" }
@@ -22,7 +22,7 @@
   end
 
   describe "#function_self_pointer_hint" do
-    subject { data.function_self_pointer_hint(function_name) }
+    subject { config.function_self_pointer_hint(function_name) }
 
     context "found in default.yml" do
       let(:function_name) { "RSTRING_PTR" }
@@ -38,7 +38,7 @@
   end
 
   describe "#should_generate_function?" do
-    subject { data.should_generate_function?(function_name) }
+    subject { config.should_generate_function?(function_name) }
 
     context "rb function (denied)" do
       let(:function_name) { "rb_check_safe_str" }
@@ -60,7 +60,7 @@
   end
 
   describe "#should_generate_struct?" do
-    subject { data.should_generate_struct?(struct_name) }
+    subject { config.should_generate_struct?(struct_name) }
 
     context "rb struct" do
       let(:struct_name) { "rb_random_struct" }
@@ -76,7 +76,7 @@
   end
 
   describe "#should_generate_type?" do
-    subject { data.should_generate_type?(type_name) }
+    subject { config.should_generate_type?(type_name) }
 
     context "rb type" do
       let(:type_name) { "rb_data_type_t" }
@@ -98,7 +98,7 @@
   end
 
   describe "#should_generate_enum?" do
-    subject { data.should_generate_enum?(enum_name) }
+    subject { config.should_generate_enum?(enum_name) }
 
     context "ruby_value_type" do
       let(:enum_name) { "ruby_value_type" }