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

Feature/generic string #1050

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

Conversation

ThibautVerron
Copy link
Contributor

Hi,

This is a pull request for the code related to the generic string delimiters ( #1039 ). You might want to squash the history, the last commits were very messy (sorry about that).

It turns out that this:

As it is, it might be slightly breaking for ruby users, for example sp-backward-sexp with %w(asd)| will now move the point to before %, not before the opening paren.

is no longer a problem.

But on the other hand, I don't know how to do it again if we were to decide that it is the wanted behaviour. Something seems to be overriding the detection of the string somewhere in sp-get-expression.

Emacs' default sexp navigation (using syntax tables) correctly navigates to the beginning of the string in the same scenario.

Thanks for your work ! :)

@ThibautVerron
Copy link
Contributor Author

You might want to squash the history, the last commits were very messy (sorry about that).

Or should I do it and offer to merge a clean branch?

@Fuco1
Copy link
Owner

Fuco1 commented Oct 19, 2020

No problem with the squash, GitHub can do it automatically when merging. Thank you for the PR, I'll review it shortly.

@Fuco1 Fuco1 self-assigned this Oct 19, 2020
@Fuco1 Fuco1 self-requested a review October 19, 2020 20:55
@Fuco1 Fuco1 removed their assignment Oct 19, 2020
@ThibautVerron
Copy link
Contributor Author

Ping? :)

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.

None yet

2 participants