From b844240212b31d9c5561c330367b8d59e1575ea9 Mon Sep 17 00:00:00 2001 From: Myriad-Dreamin <35292584+Myriad-Dreamin@users.noreply.github.com> Date: Thu, 14 Nov 2024 21:21:47 +0800 Subject: [PATCH] dev: clean up log to print (#818) --- crates/tinymist-query/src/analysis/global.rs | 3 ++- crates/tinymist-query/src/syntax/lexical_hierarchy.rs | 4 +--- crates/tinymist-query/src/upstream/complete.rs | 2 +- crates/tinymist-query/src/upstream/mod.rs | 2 +- crates/tinymist-query/src/workspace_label.rs | 3 --- crates/tinymist/src/main.rs | 8 +++++--- 6 files changed, 10 insertions(+), 12 deletions(-) diff --git a/crates/tinymist-query/src/analysis/global.rs b/crates/tinymist-query/src/analysis/global.rs index 9ebda16a..d532bc2f 100644 --- a/crates/tinymist-query/src/analysis/global.rs +++ b/crates/tinymist-query/src/analysis/global.rs @@ -1121,6 +1121,7 @@ impl AnalysisRevCache { ) -> Arc> { lg.inner.access(revision); self.manager.find_revision(revision, |slot_base| { + log::info!("analysis revision {} is created", revision.get()); slot_base .map(|e| AnalysisRevSlot { revision: e.revision, @@ -1162,7 +1163,7 @@ struct AnalysisRevSlot { impl Drop for AnalysisRevSlot { fn drop(&mut self) { - log::info!("analysis revision {} is dropped", self.revision) + log::info!("analysis revision {} is dropped", self.revision); } } diff --git a/crates/tinymist-query/src/syntax/lexical_hierarchy.rs b/crates/tinymist-query/src/syntax/lexical_hierarchy.rs index 86a1fe35..d88488c7 100644 --- a/crates/tinymist-query/src/syntax/lexical_hierarchy.rs +++ b/crates/tinymist-query/src/syntax/lexical_hierarchy.rs @@ -2,7 +2,6 @@ use std::ops::{Deref, Range}; use anyhow::anyhow; use ecow::{eco_vec, EcoString, EcoVec}; -use log::info; use lsp_types::SymbolKind; use serde::{Deserialize, Serialize}; use typst::syntax::{ @@ -42,8 +41,7 @@ pub(crate) fn get_lexical_hierarchy( worker.symbreak(); } - let e = std::time::Instant::now(); - info!("lexical hierarchy analysis took {:?}", e - b); + log::debug!("lexical hierarchy analysis took {:?}", b.elapsed()); res.map(|_| worker.stack.pop().unwrap().1) } diff --git a/crates/tinymist-query/src/upstream/complete.rs b/crates/tinymist-query/src/upstream/complete.rs index ae11e113..67b51066 100644 --- a/crates/tinymist-query/src/upstream/complete.rs +++ b/crates/tinymist-query/src/upstream/complete.rs @@ -39,7 +39,7 @@ pub fn autocomplete( ) -> Option<(usize, bool, Vec, Vec)> { let _ = complete_comments(&mut ctx) || complete_type(&mut ctx).is_none() && { - log::info!("continue after completing type"); + log::debug!("continue after completing type"); complete_labels(&mut ctx) || complete_field_accesses(&mut ctx) || complete_imports(&mut ctx) diff --git a/crates/tinymist-query/src/upstream/mod.rs b/crates/tinymist-query/src/upstream/mod.rs index 3aae1b4c..67fc0290 100644 --- a/crates/tinymist-query/src/upstream/mod.rs +++ b/crates/tinymist-query/src/upstream/mod.rs @@ -55,7 +55,7 @@ pub fn plain_docs_sentence(docs: &str) -> EcoString { let link_content = s.from(c + 1); s.eat(); - log::info!("Intra Link: {link_content}"); + log::debug!("Intra Link: {link_content}"); let link = resolve(link_content, "https://typst.app/docs/").ok(); let link = link.unwrap_or_else(|| { log::warn!("Failed to resolve link: {link_content}"); diff --git a/crates/tinymist-query/src/workspace_label.rs b/crates/tinymist-query/src/workspace_label.rs index a9498f24..d7309e73 100644 --- a/crates/tinymist-query/src/workspace_label.rs +++ b/crates/tinymist-query/src/workspace_label.rs @@ -22,8 +22,6 @@ impl SemanticRequest for WorkspaceLabelRequest { let mut symbols = vec![]; for id in ctx.source_files().clone() { - let now = reflexo::time::Instant::now(); - log::info!("workspace/label: {:?}", id); let Ok(source) = ctx.source_by_id(id) else { continue; }; @@ -39,7 +37,6 @@ impl SemanticRequest for WorkspaceLabelRequest { if let Some(mut res) = res { symbols.append(&mut res) } - log::info!("workspace/label: {:?} took {:?}", id, now.elapsed()); } Some(symbols) diff --git a/crates/tinymist/src/main.rs b/crates/tinymist/src/main.rs index 40ee7901..1d2046aa 100644 --- a/crates/tinymist/src/main.rs +++ b/crates/tinymist/src/main.rs @@ -64,10 +64,12 @@ fn main() -> anyhow::Result<()> { env_logger::builder() .filter_module("tinymist", Info) .filter_module("typst_preview", Debug) - .filter_module("typst_ts", Info) + .filter_module("typlite", Info) + .filter_module("reflexo", Info) .filter_module("sync_lsp", Info) - .filter_module("typst_ts_compiler::service::compile", Info) - .filter_module("typst_ts_compiler::service::watch", Info) + .filter_module("reflexo_typst::service::compile", Info) + .filter_module("reflexo_typst::service::watch", Info) + .filter_module("reflexo_typst::diag::console", Info) .try_init() };