Skip to content

Commit 9299cf3

Browse files
Earlopainmatzbot
authored andcommitted
[ruby/prism] Fix warn polyfill when no uplevel is provided
An unspecified uplevel is not the same as an uplevel of 1: ``` $ irb irb(main):001> warn("foo") foo => nil irb(main):002> warn("foo", uplevel: 1) /home/user/.rbenv/versions/2.7.8/lib/ruby/gems/2.7.0/gems/irb-1.14.0/lib/irb/workspace.rb:121: warning: foo => nil ``` ruby/prism@dcedd14357
1 parent 8ad5a0a commit 9299cf3

File tree

1 file changed

+16
-22
lines changed

1 file changed

+16
-22
lines changed

lib/prism/polyfill/warn.rb

Lines changed: 16 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -7,35 +7,29 @@
77
Kernel.prepend(
88
Module.new {
99
def warn(*msgs, uplevel: nil, category: nil) # :nodoc:
10-
uplevel =
11-
case uplevel
12-
when nil
13-
1
14-
when Integer
15-
uplevel + 1
16-
else
17-
uplevel.to_int + 1
18-
end
19-
20-
super(*msgs, uplevel: uplevel)
10+
case uplevel
11+
when nil
12+
super(*msgs)
13+
when Integer
14+
super(*msgs, uplevel: uplevel + 1)
15+
else
16+
super(*msgs, uplevel: uplevel.to_int + 1)
17+
end
2118
end
2219
}
2320
)
2421

2522
Object.prepend(
2623
Module.new {
2724
def warn(*msgs, uplevel: nil, category: nil) # :nodoc:
28-
uplevel =
29-
case uplevel
30-
when nil
31-
1
32-
when Integer
33-
uplevel + 1
34-
else
35-
uplevel.to_int + 1
36-
end
37-
38-
super(*msgs, uplevel: uplevel)
25+
case uplevel
26+
when nil
27+
super(*msgs)
28+
when Integer
29+
super(*msgs, uplevel: uplevel + 1)
30+
else
31+
super(*msgs, uplevel: uplevel.to_int + 1)
32+
end
3933
end
4034
}
4135
)

0 commit comments

Comments
 (0)