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 issue #5 #17

Merged
merged 2 commits into from Feb 5, 2014

Conversation

Projects
None yet
2 participants
@ghost

ghost commented Nov 1, 2012

This is an attempt to fix issue #5 by defining a new motion surround-line to be used instead of evil-line. Hope it helps.

Frank Fischer added some commits Nov 1, 2012

Use `surround-line' instead of `evil-line' for linewise operations
`evil-line' is a motion that covers a number of lines as a line
range. This causes surround to add additional line breaks before and
after the text to be wrapped if used with `yss' commands.

The function `surround-line' covers the same characters but this time
as exclusive character range.

greduan commented Jan 1, 2014

@lyro Can you confirm it works right? @timcharper could you merge?

@ghost

ghost commented Jan 2, 2014

Well, it seems to work for me, but since I'm probably the only one using
this patch there may be very well some bugs that just do not occur in
may workflow.

Am 01.01.2014 21:27, schrieb Eduán Lávaque:

@lyro Can you confirm it works right? @timcharper could you merge?


Reply to this email directly or view it on GitHub:
timcharper#17 (comment)

@timcharper timcharper merged commit 88f371a into emacs-evil:master Feb 5, 2014

Collaborator

timcharper commented Feb 5, 2014

I don't see any problems, and things still work for me. Given this, and that it's been a month and neither of you two seem to have discovered any issues, it seems safe to merge. Thank you!

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