break throws NoMethodError on master

[caius]

I just installed pry-byebug from master (17f9bc7 specifically) and found the following:

$ pry
2.1.2 (main):0 > require "pry-byebug"
=> false
2.1.2 (main):0 > class Fred; def foo; end; end
=> :foo
2.1.2 (main):0 > break Fred#foo
NoMethodError: undefined method `<<' for nil:NilClass
from /Users/caius/.gem/ruby/2.1.2/gems/byebug-3.2.0/lib/byebug.rb:44:in `add_breakpoint'

The same code works fine on v1.3.3 installed from rubygems.

[caius]

It also appears to happen when specifying a line with file:linenumber:

2.1.2 (main):0 > break /tmp/fred.rb:2
NoMethodError: undefined method `<<' for nil:NilClass
from /Users/caius/.gem/ruby/2.1.2/gems/byebug-3.2.0/lib/byebug.rb:44:in `add_breakpoint'

[deivid-rodriguez]

Can you try again using byebug's last master as well? Thanks!!

[caius]

I can set the breakpoint ok with byebug's master now; but I have to call Byebug.start before it activates the breakpoint and actually drops to a debugger - is that expected? I think that's a break with the behaviour of 3.2.0 (but I'm not sure on that.)

[deivid-rodriguez]

I've added your workaround to latest master. Does it work now?

[caius]

Yippie! Works as I expect with previous behaviour, and by not exiting on continue being used.

$ pry
2.1.2 (main):0 > require "pry-byebug"
=> false
2.1.2 (main):0 > class Fred; def foo; end; end
=> :foo
2.1.2 (main):0 > break Fred#foo
Breakpoint 1: Fred#foo (Enabled) :

2: class Fred; def foo; end; end

2.1.2 (main):0 > Fred.new.foo

Breakpoint 1. First hit.

From: (pry) @ line 2 Fred#foo:

 => 2: class Fred; def foo; end; end

2.1.2 (#<Fred:0x007fd616e230a8>):1 > continue
=> nil
2.1.2 (main):0 > continue # Just for giggles
Error: Cannot find local context. Did you use `binding.pry`?

Many thanks for adding this 😄 👍💯

[deivid-rodriguez]

Awesome @caius, you are welcome!! And thanks for the testing & quick feedback!!