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.

Sign up for GitHub

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

Jump to bottom

feat(commands) increment by range #4418

Merged
merged 9 commits into from
Nov 11, 2022
Merged

feat(commands) increment by range #4418

Changes from 4 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
32644a9
feat(commands) increment by range
Bruce-Hopkins Oct 22, 2022
455c46c
added increasing increment as a new keymap
Bruce-Hopkins Oct 22, 2022
ee146f4
Changed keymapping for increasing increment
Bruce-Hopkins Oct 25, 2022
0ac55c0
Changed range increment to work under a special register
Bruce-Hopkins Oct 31, 2022
5b5d0a7
Added enum for increment feature
Bruce-Hopkins Nov 3, 2022
c5c59f3
removed amount parameter from increment_impl
Bruce-Hopkins Nov 6, 2022
d949f6e
cleaned increment code
Bruce-Hopkins Nov 7, 2022
76ff510
removed enum numbers
Bruce-Hopkins Nov 7, 2022
fdd5e4b
formatted with cargo fmt
Bruce-Hopkins Nov 11, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 19 additions & 3 deletions helix-term/src/commands.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4808,14 +4808,27 @@ fn add_newline_impl(cx: &mut Context, open: Open) {
apply_transaction(&transaction, doc, view);
}

/// If the register is '#', then it will return 1. Otherwise it will be 0
fn get_increase_by_value(cx: &mut Context) -> i64 {
match cx.register {
Some(value) => match value.eq(&'#') {
true => 1,
false => 0,
},
None => 0,
}
}

/// Increment object under cursor by count.
fn increment(cx: &mut Context) {
increment_impl(cx, cx.count() as i64);
let increase_by = get_increase_by_value(cx);
increment_impl(cx, cx.count() as i64, increase_by);
}

/// Decrement object under cursor by count.
fn decrement(cx: &mut Context) {
increment_impl(cx, -(cx.count() as i64));
let decrease_by = -get_increase_by_value(cx);
increment_impl(cx, -(cx.count() as i64), decrease_by);
}

/// This function differs from find_next_char_impl in that it stops searching at the newline, but also
Expand All @@ -4839,7 +4852,7 @@ fn find_next_char_until_newline<M: CharMatcher>(
}

/// Decrement object under cursor by `amount`.
fn increment_impl(cx: &mut Context, amount: i64) {
fn increment_impl(cx: &mut Context, amount: i64, increase_by: i64) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of passing amounts to increase_impl, we can create an enum

enum IncrementDirection {
    Increase,
    Decrease,
}

And pass that to increment_impl, then check the cx.count and cx.register in the body of increment_impl and then decide whether to use a positive or negative amount based on the passed direction.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, won't we need a 3rd value for the enum if we don't want to increase it at all?

And pass that to increment_impl, then check the cx.count and cx.register in the body of increment_impl and then decide whether to use a positive or negative amount based on the passed direction.

Alright, that makes sense.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Count can't be 0 so we don't need a third member

// TODO: when incrementing or decrementing a number that gets a new digit or lose one, the
// selection is updated improperly.
find_char_impl(
Expand All @@ -4855,6 +4868,7 @@ fn increment_impl(cx: &mut Context, amount: i64) {
let selection = doc.selection(view.id);
let text = doc.text().slice(..);

let mut amount = amount;
let changes: Vec<_> = selection
.ranges()
.iter()
Expand All @@ -4870,6 +4884,8 @@ fn increment_impl(cx: &mut Context, amount: i64) {

let (range, new_text) = incrementor.increment(amount);

amount += increase_by;

Some((range.from(), range.to(), Some(new_text)))
})
.collect();
Expand Down