chore: cleanup

Author: matoous

helix-term/src/commands.rs

@@ -433,7 +433,7 @@ impl MappableCommand {
         add_newline_below, "Add newline below",
         goto_type_definition, "Goto type definition",
         goto_implementation, "Goto implementation",
-        request_code_lenses, "Request code lenses",
+        show_code_lenses_under_cursor, "Show code lenses under cursor",
         goto_file_start, "Goto line number <n> else file start",
         goto_file_end, "Goto file end",
         extend_to_file_start, "Extend to line number<n> else file start",

helix-term/src/commands/lsp.rs

@@ -1467,9 +1467,7 @@ impl ui::menu::Item for lsp::CodeLens {
     }
 }
 
-// TODO: should be run the same way as diagnostic - shouldn't require manual
-// trigger to set lenses.
-pub fn request_code_lenses(cx: &mut Context) {
+pub fn show_code_lenses_under_cursor(cx: &mut Context) {
     let (_view, doc) = current!(cx.editor);
     let doc_id = doc.id();
     let language_server =

helix-term/src/handlers/code_lenses.rs

@@ -12,11 +12,16 @@ use tokio::time::Instant;
 
 use crate::job;
 
+// TODO: use to update code lenses positions on document changes
 #[derive(Default)]
+#[allow(dead_code)]
 pub(super) struct DocumentCodeLensesHandler {
     docs: HashSet<DocumentId>,
 }
 
+// TODO: use to update debounce document changes when udpating positions of code lenses
+// TODO: share with color swatches and possibly other annotations
+#[allow(dead_code)]
 const DOCUMENT_CHANGE_DEBOUNCE: Duration = Duration::from_millis(250);
 
 impl helix_event::AsyncHook for DocumentCodeLensesHandler {

helix-term/src/keymap/default.rs

@@ -236,7 +236,7 @@ pub fn default() -> HashMap<Mode, KeyTrie> {
             "g" => changed_file_picker,
             "a" => code_action,
             "'" => last_picker,
-            "L" => request_code_lenses,
+            "l" => show_code_lenses_under_cursor,
             "G" => { "Debug (experimental)" sticky=true
                 "l" => dap_launch,
                 "r" => dap_restart,