From 0c8f0c0334d449dd71928a697cfba0207be74a63 Mon Sep 17 00:00:00 2001 From: Zi How Poh Date: Tue, 28 Jan 2025 04:41:34 +0800 Subject: [PATCH] goto_diag: avoid wraparound by default (#12704) --- helix-term/src/commands.rs | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs index 2a511cbeb..db912e6c0 100644 --- a/helix-term/src/commands.rs +++ b/helix-term/src/commands.rs @@ -3824,8 +3824,7 @@ fn goto_next_diag(cx: &mut Context) { let diag = doc .diagnostics() .iter() - .find(|diag| diag.range.start > cursor_pos) - .or_else(|| doc.diagnostics().first()); + .find(|diag| diag.range.start > cursor_pos); let selection = match diag { Some(diag) => Selection::single(diag.range.start, diag.range.end), @@ -3852,8 +3851,7 @@ fn goto_prev_diag(cx: &mut Context) { .diagnostics() .iter() .rev() - .find(|diag| diag.range.start < cursor_pos) - .or_else(|| doc.diagnostics().last()); + .find(|diag| diag.range.start < cursor_pos); let selection = match diag { // NOTE: the selection is reversed because we're jumping to the