Skip to content

Commit 155a1ad

Browse files
committed
Initialize dist_preprocessed_header_file with default value
1 parent e54d639 commit 155a1ad

File tree

3 files changed

+10
-5
lines changed

3 files changed

+10
-5
lines changed

lib/ruby_header_parser.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# frozen_string_literal: true
22

33
require "yaml"
4+
require "tmpdir"
45

56
require_relative "ruby_header_parser/version"
67

lib/ruby_header_parser/parser.rb

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,18 @@ class Parser # rubocop:disable Metrics/ClassLength
2626
RbConfig::CONFIG["rubyhdrdir"],
2727
].freeze
2828

29-
# @param header_file [String] path to ruby.h
29+
# @param header_file [String] Path to `ruby.h`
3030
# @param include_paths [Array<String>]
31-
# @param dist_preprocessed_header_file [String]
32-
def initialize(dist_preprocessed_header_file:, header_file: DEFAULT_HEADER_FILE,
31+
# @param dist_preprocessed_header_file [String,nil] Destination path to the output of preprocessed ruby.h.
32+
# (default: `"#{Dir.tmpdir}/ruby_preprocessed.h"`)
33+
#
34+
# @note `dist_preprocessed_header_file` is used as the output destination for temporary files when the parser
35+
# is executed
36+
def initialize(dist_preprocessed_header_file: nil, header_file: DEFAULT_HEADER_FILE,
3337
include_paths: DEFAULT_INCLUDE_PATHS)
3438
@header_file = header_file
3539
@include_paths = include_paths
36-
@dist_preprocessed_header_file = dist_preprocessed_header_file
40+
@dist_preprocessed_header_file = dist_preprocessed_header_file || File.join(Dir.tmpdir, "ruby_preprocessed.h")
3741
@data = Data.new
3842
end
3943

sig/ruby_header_parser/parser.rbs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ module RubyHeaderParser
99
DEFAULT_INCLUDE_PATHS: Array[String]
1010

1111
def initialize: (
12-
dist_preprocessed_header_file: String,
12+
?dist_preprocessed_header_file: String?,
1313
?header_file: String,
1414
?include_paths: Array[String]
1515
) -> void

0 commit comments

Comments
 (0)