-
Notifications
You must be signed in to change notification settings - Fork 716
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
src: Select the data inserted by !
and <a-!>
#4335
Conversation
I've been using this for a bit, it feels nice. |
I am working on a refactoring of selections insert/replace that should make implementing this much cleaner, I do agree with this change (and the p/P one) but want to finish my refactoring first. |
07bc206
to
9342991
Compare
Rebased on master. |
9342991
to
a4a0336
Compare
Rebased on master. |
a4a0336
to
a26b473
Compare
I intend to merge this, I just want to make a release before going into various breaking changes |
As a release has been done a couple of weeks after the last message (https://github.com/mawww/kakoune/releases/tag/v2021.11.08), should this PR be merged now? |
Commit 85b78dd (src: Select the data inserted by `!` and `<a-!>`, merged on 2021-03-06) broke autorestore by making it delete the restored content. I've been using it for 6 months but never noticed since I didn't use autorestore Reproducer: HOME=$PWD kak -s foo README.asciidoc -e 'exec iUNSAVED-CONTENT' # In another terminal: ps aux | awk '/kak -s foo/ {print $2; exit}' | xargs kill -HUP HOME=$PWD kak -s foo README.asciidoc Delete the trailing newline instead of the restored content. While at it, remove some <space> commands from execute-keys, to make it work on the breaking-cleanups branch which swaps <space> and ",". Closes mawww#4335
Closes #1468