Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

fix show command #99

Merged
merged 1 commit into from Sep 23, 2013

Conversation

Projects
None yet
2 participants

show command is broken on master after d7f5ea7

stacktrace

rdb:1) show
INTERNAL ERROR!!! undefined local variable or method `subcmd' for #<Debugger::ShowCommand:0x007ffeb0f4bf80>
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/commands/show.rb:185:in `execute'
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/processor.rb:279:in `one_cmd'
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/processor.rb:265:in `block (2 levels) in process_commands'
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/processor.rb:264:in `each'
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/processor.rb:264:in `block in process_commands'
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/processor.rb:257:in `catch'
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/processor.rb:257:in `process_commands'
        /opt/boxen/rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/debugger-1.6.0/lib/ruby-debug/processor.rb:190:in `at_line'
        (eval):5:in `block in at_line'
        <internal:prelude>:10:in `synchronize'

This patch fixes the problem and add a regression test.

@cldwalker cldwalker added a commit that referenced this pull request Sep 23, 2013

@cldwalker cldwalker Merge pull request #99 from arthurnn/show_command
fix show command
5c714da

@cldwalker cldwalker merged commit 5c714da into cldwalker:master Sep 23, 2013

1 check passed

default The Travis CI build passed
Details
Owner

cldwalker commented Sep 23, 2013

Nice catch. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment