kc/helix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

refactor: remove nesting

Browse source
This commit is contained in:
Nik Revenco 2025-03-19 12:49:37 +00:00
parent 215fb29fe6
commit fc19b756ee
1 changed files with 26 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

49
helix-term/src/handlers/blame.rs
View file

@ -64,31 +64,34 @@ impl helix_event::AsyncHook for BlameHandler {
pub(super) fn register_hooks(handlers: &Handlers) {
let tx = handlers.blame.clone();
register_hook!(move |event: &mut PostCommand<'_, '_>| {
if event.cx.editor.config().vcs.blame {
let (view, doc) = current!(event.cx.editor);
let text = doc.text();
let selection = doc.selection(view.id);
let Some(file) = doc.path() else {
return Ok(());
};
let file = file.to_path_buf();
let Ok(cursor_line) = TryInto::<u32>::try_into(
text.char_to_line(selection.primary().cursor(doc.text().slice(..))),
) else {
return Ok(());
};
send_blocking(
&tx,
BlameEvent::PostCommand {
file,
cursor_line,
diff_providers: event.cx.editor.diff_providers.clone(),
},
);
if !event.cx.editor.config().vcs.blame {
return Ok(());
}
let (view, doc) = current!(event.cx.editor);
let text = doc.text();
let selection = doc.selection(view.id);
let Some(file) = doc.path() else {
return Ok(());
};
let file = file.to_path_buf();
let Ok(cursor_line) =
u32::try_from(text.char_to_line(selection.primary().cursor(doc.text().slice(..))))
else {
return Ok(());
};
send_blocking(
&tx,
BlameEvent::PostCommand {
file,
cursor_line,
// ok to clone because diff_providers is very small
diff_providers: event.cx.editor.diff_providers.clone(),
},
);
Ok(())
});
}