diff options
author | Gabriel Hansson | 2023-07-09 20:50:24 +0000 |
---|---|---|
committer | GitHub | 2023-07-09 20:50:24 +0000 |
commit | c1488267e59a341ece2a8bb274e66127dca72088 (patch) | |
tree | 9f87c8f33b9e5679743b2fbac75c9a39688dba4f /flake.lock | |
parent | 9a324f337a2705e68035a356ad6b626ee4c6669b (diff) |
(Updated) Apply motion API refinements (#6078)
* _apply_motion generalization where possible
API encourages users to not forget setting `editor.last_motion` when
applying a motion. But also not setting `last_motion` without applying a
motion first.
* (rename) will_find_char -> find_char
method name makes it sound like it would be returning a boolean.
* use _apply_motion in find_char
Feature that falls out from this is that repetitions of t,T,f,F are
saved with the context extention/move and count. (Not defaulting to extend
by 1 count).
* Finalize apply_motion API
last_motion is now a private field and can only be set by calling
Editor.apply_motion(). Removing need (and possibility) of writing:
`motion(editor); editor.last_motion = motion`
Now it's just: `editor.apply_motion(motion)`
* editor.last_message: rm Box wrap around Arc
* Use pre-existing `Direction` rather than custom `SearchDirection`.
* `LastMotion` type alias for `Option<Arc<dyn Fn(&mut Editor)>>`
* Take motion rather than cloning it.
Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
* last_motion as Option<Motion>.
* Use `Box` over `Arc` for `last_motion`.
---------
Co-authored-by: Michael Davis <mcarsondavis@gmail.com>
Diffstat (limited to 'flake.lock')
0 files changed, 0 insertions, 0 deletions