Skip to content
This repository was archived by the owner on Dec 29, 2022. It is now read-only.

Commit 577ab7d

Browse files
committed
Apply clippy lints
1 parent 1256334 commit 577ab7d

File tree

4 files changed

+7
-5
lines changed

4 files changed

+7
-5
lines changed

rls-rustc/src/lib.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,6 @@ impl Callbacks for ShimCalls {
120120

121121
let sess = compiler.session();
122122
let input = compiler.input();
123-
let crate_name = queries.crate_name().unwrap().peek().clone();
124123

125124
let cwd = &sess.working_dir.0;
126125

@@ -131,7 +130,7 @@ impl Callbacks for ShimCalls {
131130
.and_then(|path| src_path(Some(cwd), path));
132131

133132
let krate = Crate {
134-
name: crate_name.to_owned(),
133+
name: queries.crate_name().unwrap().peek().to_owned(),
135134
src_path,
136135
disambiguator: sess.local_crate_disambiguator().to_fingerprint().as_value(),
137136
edition: match sess.edition() {

rls/src/actions/mod.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -491,12 +491,13 @@ fn find_word_at_pos(line: &str, pos: Column) -> (Column, Column) {
491491
.map(|(i, _)| i + 1)
492492
.unwrap_or(0) as u32;
493493

494+
#[allow(clippy::filter_next)]
494495
let end = line
495496
.chars()
496497
.enumerate()
497498
.skip(col)
498499
.filter(|&(_, c)| !is_ident_char(c))
499-
.nth(0)
500+
.next()
500501
.map(|(i, _)| i)
501502
.unwrap_or(col) as u32;
502503

rls/src/actions/post_build.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,9 @@ impl RefUnwindSafe for AnalysisQueue {}
291291

292292
impl Drop for AnalysisQueue {
293293
fn drop(&mut self) {
294-
let _ = self.queue.lock().map(|mut q| q.push(QueuedJob::Terminate));
294+
if let Ok(mut queue) = self.queue.lock() {
295+
queue.push(QueuedJob::Terminate);
296+
}
295297
}
296298
}
297299

rls/src/build/plan.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ impl JobQueue {
176176
&internals.vfs,
177177
&args,
178178
job.get_envs(),
179-
job.get_cwd().or_else(|| cwd.as_ref().map(|p| &**p)),
179+
job.get_cwd().or_else(|| cwd.as_deref()),
180180
&build_dir,
181181
Arc::clone(&internals.config),
182182
&internals.env_lock.as_facade(),

0 commit comments

Comments
 (0)