[IMP] address compiler warnings

Author: mmahrouss

server/src/core/import_resolver.rs

@@ -1,10 +1,8 @@
 use glob::glob;
-use itertools::Itertools;
-use lsp_types::{CompletionItemKind, Diagnostic, DiagnosticTag, Position, Range};
+use lsp_types::{Diagnostic, DiagnosticTag, Position, Range};
 use ruff_python_ast::name::Name;
-use serde::Serialize;
 use tracing::error;
-use std::collections::{HashMap, HashSet};
+use std::collections::HashMap;
 use std::rc::Rc;
 use std::cell::RefCell;
 use std::path::{Path, PathBuf};
@@ -541,7 +539,7 @@ pub fn get_all_valid_names(session: &mut SessionInfo, source_file_symbol: &Rc<Re
     result
 }
 
-fn valid_names_for_a_symbol(session: &mut SessionInfo, symbol: &Rc<RefCell<Symbol>>, start_filter: &OYarn, only_on_disk: bool) -> HashMap<OYarn, SymType> {
+fn valid_names_for_a_symbol(_session: &mut SessionInfo, symbol: &Rc<RefCell<Symbol>>, start_filter: &OYarn, only_on_disk: bool) -> HashMap<OYarn, SymType> {
     let mut res = HashMap::new();
     match symbol.borrow().typ() {
         SymType::FILE => {

server/src/core/symbols/symbol.rs

@@ -1120,7 +1120,7 @@ impl Symbol {
         }
     }
 
-    pub fn iter_symbols(&self) -> std::collections::hash_map::Iter<OYarn, HashMap<u32, Vec<Rc<RefCell<Symbol>>>>> {
+    pub fn iter_symbols(&self) -> std::collections::hash_map::Iter<'_, OYarn, HashMap<u32, Vec<Rc<RefCell<Symbol>>>>> {
         match self {
             Symbol::File(f) => {
                 f.symbols.iter()

server/src/features/ast_utils.rs

@@ -6,13 +6,13 @@ use crate::core::evaluation::{AnalyzeAstResult, Context, ContextValue, Evaluatio
 use crate::core::odoo::SyncOdoo;
 use crate::core::import_resolver::{resolve_from_stmt, resolve_import_stmt};
 use crate::core::symbols::symbol::Symbol;
-use crate::core::file_mgr::{FileInfo, FileInfoAst};
+use crate::core::file_mgr::FileInfoAst;
 use crate::threads::SessionInfo;
 use crate::S;
 use ruff_python_ast::name::Name;
 use ruff_python_ast::visitor::{Visitor, walk_expr, walk_stmt, walk_alias, walk_except_handler, walk_parameter, walk_keyword, walk_pattern_keyword, walk_type_param, walk_pattern};
 use ruff_python_ast::{Alias, AtomicNodeIndex, ExceptHandler, Expr, ExprCall, Identifier, Keyword, Parameter, Pattern, PatternKeyword, Stmt, TypeParam};
-use ruff_text_size::{Ranged, TextRange, TextSize, TextSlice};
+use ruff_text_size::{Ranged, TextRange, TextSize};
 use tracing::warn;
 
 pub struct AstUtils {}

server/src/features/hover.rs

@@ -42,7 +42,7 @@ impl HoverFeature {
         if let Ok(document) = document {
             let root = document.root_element();
             let (symbols, range) = XmlAstUtils::get_symbols(session, file_symbol, root, offset, true);
-            let range = range.map(|r| (file_info.borrow().std_range_to_range(&r)));
+            let range = range.map(|r| file_info.borrow().std_range_to_range(&r));
             let evals = symbols.iter().filter(|s| matches!(s, XmlAstResult::SYMBOL(_)))
                 .map(|s| Evaluation::eval_from_symbol(&Rc::downgrade(&s.as_symbol()), Some(false))).collect::<Vec<Evaluation>>();
             return Some(Hover { contents:

server/src/main.rs

@@ -8,8 +8,9 @@ use tracing_panic::panic_hook;
 use tracing_subscriber::{fmt, FmtSubscriber, layer::SubscriberExt};
 
 use std::{env, path::PathBuf, process};
-use odoo_ls_server::allocator::TrackingAllocator;
 
+#[cfg(debug_assertions)]
+use odoo_ls_server::allocator::TrackingAllocator;
 #[cfg(debug_assertions)]
 #[global_allocator]
 static GLOBAL: TrackingAllocator = TrackingAllocator;

server/src/server.rs

@@ -1,4 +1,4 @@
-use std::{io::Error, panic, sync::{self, Arc, Mutex, atomic::AtomicBool}, thread::JoinHandle};
+use std::{io::Error, panic, sync::{Arc, Mutex, atomic::AtomicBool}, thread::JoinHandle};
 
 use crossbeam_channel::{Receiver, Select, Sender};
 use lsp_server::{Connection, IoThreads, Message, ProtocolError, RequestId, ResponseError};