Skip to content

Commit c65c970

Browse files
committed
fix b Kernel#foo message
For the exsiting class but no method error message was strange.
1 parent ac5fc97 commit c65c970

File tree

2 files changed

+13
-8
lines changed

2 files changed

+13
-8
lines changed

lib/debug/breakpoint.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -421,7 +421,7 @@ def to_s
421421
end
422422

423423
class MethodBreakpoint < Breakpoint
424-
attr_reader :sig_method_name, :method
424+
attr_reader :sig_method_name, :method, :klass
425425

426426
def initialize b, klass_name, op, method_name, cond: nil, command: nil, path: nil
427427
@sig_klass_name = klass_name

lib/debug/thread_client.rb

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -762,15 +762,20 @@ def make_breakpoint args
762762
begin
763763
bp.enable
764764
rescue NameError => e
765-
if constant_name? klass_name
766-
puts "Unknown constant name: \"#{e.name}\""
767-
# TODO: Ractor support
768-
Session.activate_method_added_trackers
765+
if bp.klass
766+
puts "Unknown method name: \"#{e.name}\""
769767
else
770-
# only Class name is allowed
771-
puts "Not a constant name: \"#{klass_name}\""
772-
bp = nil
768+
# klass_name can not be evaluated
769+
if constant_name? klass_name
770+
puts "Unknown constant name: \"#{e.name}\""
771+
else
772+
# only Class name is allowed
773+
puts "Not a constant name: \"#{klass_name}\""
774+
bp = nil
775+
end
773776
end
777+
778+
Session.activate_method_added_trackers if bp
774779
rescue Exception => e
775780
puts e.inspect
776781
bp = nil

0 commit comments

Comments
 (0)