Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

set :end correctly when no next sexp in sp-get-hybrid-sexp #975

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

edam
Copy link
Contributor

@edam edam commented Apr 29, 2019

This seems to work for me.

The logic is that when sp-forward-sexp (in sp--get-hybrid-sexp-end) returns a next sexp whose :beg is before point, it must be because there are no more and it returned the parent sexp. And so, when :beg < point, either the end of the line or the end of the parent sexp, whichever is sooner.

@edam
Copy link
Contributor Author

edam commented Apr 29, 2019

Fixes #974.

@edam
Copy link
Contributor Author

edam commented Jul 15, 2022

Tests now pass as well as they do without this patch! (i.e., diff of ert-runner is identical, with and without these changes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

2 participants