#1 Can't set a breakpoint in narrowed buffer

Open
opened 8 months ago by nico · 2 comments
nico commented 8 months ago

I didn't try to reproduce the bug, but I think that

(indium-backend-add-breakpoint (indium-backend)
                                     buffer-file-name
                                     (1- (line-number-at-pos))
                                     (lambda (line id condition)
                                       (indium-breakpoint-added id ov))
                                     condition)

in indium-breakpoint-add can't work in narrowed buffers because line-number-at-pos return the narrowed line number. I submitted a patch to Emacs, but it hasn't been merged yet :-).


Issue migrated from Gitlab

  • Original author: @DamienCassou
  • Created: 2017-06-16T15:46:03.955Z
  • Updated: 2017-06-19T09:31:04.501Z
I didn't try to reproduce the bug, but I think that ```common-lisp (indium-backend-add-breakpoint (indium-backend) buffer-file-name (1- (line-number-at-pos)) (lambda (line id condition) (indium-breakpoint-added id ov)) condition) ``` in `indium-breakpoint-add` can't work in narrowed buffers because `line-number-at-pos` return the narrowed line number. I submitted a patch to Emacs, but it hasn't been merged yet :-). ----- #### Issue migrated from Gitlab - Original author: @DamienCassou - Created: 2017-06-16T15:46:03.955Z - Updated: 2017-06-19T09:31:04.501Z
nico commented 8 months ago
Owner

changed the description


Comment migrated from Gitlab

  • Original author: @DamienCassou
  • Created: 2017-06-16T15:47:40.717Z
  • Updated: 2017-06-16T15:47:40.717Z
changed the description ----- #### Comment migrated from Gitlab - Original author: @DamienCassou - Created: 2017-06-16T15:47:40.717Z - Updated: 2017-06-16T15:47:40.717Z
nico commented 8 months ago
Owner

It's been merged in Emacs master :rocket:


Comment migrated from Gitlab

  • Original author: @nico
  • Created: 2017-06-19T09:31:04.461Z
  • Updated: 2017-06-19T09:31:04.461Z
It's been merged in Emacs master :rocket: ----- #### Comment migrated from Gitlab - Original author: @nico - Created: 2017-06-19T09:31:04.461Z - Updated: 2017-06-19T09:31:04.461Z
Sign in to join this conversation.
No Label
No Milestone
No Assignees
1 Participants
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
Cancel
Save
There is no content yet.