Skip to content

Commit 87a71ac

Browse files
committed
Make uplevel suitable as the option to Kernel#warn
Make Gem::BUNDLED_GEMS.uplevel returns `nil` if `require` frame is not found, for the simplicity.
1 parent 90abfe6 commit 87a71ac

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

lib/bundled_gems.rb

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,12 +49,7 @@ def self.replace_require(specs)
4949
kernel_class.send(:alias_method, :no_warning_require, :require)
5050
kernel_class.send(:define_method, :require) do |name|
5151
if message = ::Gem::BUNDLED_GEMS.warning?(name, specs: spec_names)
52-
uplevel = ::Gem::BUNDLED_GEMS.uplevel
53-
if uplevel > 0
54-
Kernel.warn message, uplevel: uplevel
55-
else
56-
Kernel.warn message
57-
end
52+
Kernel.warn message, uplevel: ::Gem::BUNDLED_GEMS.uplevel
5853
end
5954
kernel_class.send(:no_warning_require, name)
6055
end
@@ -90,7 +85,7 @@ def self.uplevel
9085
break
9186
end
9287
end
93-
require_found ? 1 : frame_count - 1
88+
require_found ? 1 : frame_count > 1 ? frame_count - 1 : nil
9489
end
9590

9691
def self.warning?(name, specs: nil)

0 commit comments

Comments
 (0)