From 623dbe7a5786a5aa600b14a7e13e26777313d844 Mon Sep 17 00:00:00 2001 From: David Sherret Date: Wed, 30 Nov 2022 14:19:32 -0500 Subject: chore: remove unnecessary lifetimes (#16878) It seems we don't really need to allow these clippy rules. --- cli/lsp/documents.rs | 2 +- core/runtime.rs | 8 ++++---- tools/lint.js | 7 ------- 3 files changed, 5 insertions(+), 12 deletions(-) diff --git a/cli/lsp/documents.rs b/cli/lsp/documents.rs index 1fd9b541d..fb0540bcd 100644 --- a/cli/lsp/documents.rs +++ b/cli/lsp/documents.rs @@ -634,7 +634,7 @@ struct FileSystemDocuments { } impl FileSystemDocuments { - pub fn get<'a>( + pub fn get( &mut self, cache: &HttpCache, maybe_resolver: Option<&dyn deno_graph::source::Resolver>, diff --git a/core/runtime.rs b/core/runtime.rs index 4647160c9..7ce9d3d09 100644 --- a/core/runtime.rs +++ b/core/runtime.rs @@ -1279,8 +1279,8 @@ impl JsRuntime { } } -fn get_stalled_top_level_await_message_for_module<'s>( - scope: &'s mut v8::HandleScope, +fn get_stalled_top_level_await_message_for_module( + scope: &mut v8::HandleScope, module_id: ModuleId, ) -> Vec> { let module_map = JsRuntime::module_map(scope); @@ -1296,8 +1296,8 @@ fn get_stalled_top_level_await_message_for_module<'s>( messages } -fn find_stalled_top_level_await<'s>( - scope: &'s mut v8::HandleScope, +fn find_stalled_top_level_await( + scope: &mut v8::HandleScope, ) -> Vec> { let module_map = JsRuntime::module_map(scope); let module_map = module_map.borrow(); diff --git a/tools/lint.js b/tools/lint.js index 4e842c2b9..a9279bb25 100755 --- a/tools/lint.js +++ b/tools/lint.js @@ -117,13 +117,6 @@ async function clippy() { "--", "-D", "warnings", - "-A", - // https://github.com/rust-lang/rust-clippy/issues/407 - "clippy::extra_unused_lifetimes", - "-A", - // https://github.com/rust-lang/rust-clippy/issues/7271 - // False positives in core/resources.rs for lifetime elision. - "clippy::needless_lifetimes", ], stdout: "inherit", stderr: "inherit", -- cgit v1.2.3