Skip to content

Commit 591e2fe

Browse files
Merge pull request rust-lang#22815 from ada4a/push-mqsknqnotoku
internal: remove manual lifetime extension
2 parents b0fec0d + 23df9bb commit 591e2fe

5 files changed

Lines changed: 9 additions & 35 deletions

File tree

crates/hir-def/src/nameres/mod_resolution.rs

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -149,13 +149,7 @@ impl DirPath {
149149
if attr.starts_with("./") {
150150
attr = &attr["./".len()..];
151151
}
152-
let tmp;
153-
let attr = if attr.contains('\\') {
154-
tmp = attr.replace('\\', "/");
155-
&tmp
156-
} else {
157-
attr
158-
};
152+
let attr = if attr.contains('\\') { &attr.replace('\\', "/") } else { attr };
159153
let res = format!("{base}{attr}");
160154
res
161155
}

crates/hir-def/src/nameres/path_resolution.rs

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -504,12 +504,10 @@ impl DefMap {
504504
);
505505
}
506506

507-
let def_map;
508507
let module_data = if module.block(db) == self.block_id() {
509508
&self[module]
510509
} else {
511-
def_map = module.def_map(db);
512-
&def_map[module]
510+
&module.def_map(db)[module]
513511
};
514512

515513
// Since it is a qualified path here, it should not contains legacy macros
@@ -753,14 +751,7 @@ impl DefMap {
753751

754752
fn resolve_in_prelude(&self, db: &dyn SourceDatabase, name: &Name) -> PerNs {
755753
if let Some((prelude, _use)) = self.prelude {
756-
let keep;
757-
let def_map = if prelude.krate(db) == self.krate {
758-
self
759-
} else {
760-
// Extend lifetime
761-
keep = prelude.def_map(db);
762-
keep
763-
};
754+
let def_map = if prelude.krate(db) == self.krate { self } else { prelude.def_map(db) };
764755
def_map[prelude].scope.get(name)
765756
} else {
766757
PerNs::none()

crates/ide/src/hover/render.rs

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -326,13 +326,7 @@ pub(super) fn try_for_lint(attr: &ast::Attr, token: &SyntaxToken) -> Option<Hove
326326
_ => return None,
327327
};
328328

329-
let tmp;
330-
let needle = if is_clippy {
331-
tmp = format!("clippy::{}", token.text());
332-
&tmp
333-
} else {
334-
token.text()
335-
};
329+
let needle = if is_clippy { &format!("clippy::{}", token.text()) } else { token.text() };
336330

337331
let lint =
338332
lints.binary_search_by_key(&needle, |lint| lint.label).ok().map(|idx| &lints[idx])?;

crates/ide/src/inlay_hints/closing_brace.rs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,18 +75,17 @@ pub(super) fn hints(
7575
node = node.parent()?;
7676

7777
let parent = label.syntax().parent()?;
78-
let block;
79-
match_ast! {
78+
let block = match_ast! {
8079
match parent {
8180
ast::BlockExpr(block_expr) => {
82-
block = block_expr.stmt_list()?;
81+
block_expr.stmt_list()?
8382
},
8483
ast::AnyHasLoopBody(loop_expr) => {
85-
block = loop_expr.loop_body()?.stmt_list()?;
84+
loop_expr.loop_body()?.stmt_list()?
8685
},
8786
_ => return None,
8887
}
89-
}
88+
};
9089
closing_token = block.r_curly_token()?;
9190

9291
let lifetime = label.lifetime()?.to_string();

xtask/src/install.rs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -108,12 +108,8 @@ fn install_client(sh: &Shell, client_opt: ClientOpt) -> anyhow::Result<()> {
108108
};
109109

110110
// Find the appropriate VS Code binary.
111-
let lifetime_extender;
112111
let candidates: &[&str] = match client_opt.code_bin.as_deref() {
113-
Some(it) => {
114-
lifetime_extender = [it];
115-
&lifetime_extender[..]
116-
}
112+
Some(it) => &[it],
117113
None => VS_CODES,
118114
};
119115
let code = candidates

0 commit comments

Comments
 (0)