Skip to content

Commit

Permalink
Auto merge of #16028 - Young-Flash:fix-issue-16012, r=HKalbasi
Browse files Browse the repository at this point in the history
fix: make drop inlay hint more readable

![drop_inlay_hint](https://github.com/rust-lang/rust-analyzer/assets/71162630/bb18707f-3278-435d-a938-ccff4c685586)

follow up #16000, close #16012
  • Loading branch information
bors committed Dec 7, 2023
2 parents 4196675 + afc4075 commit bc9c952
Showing 1 changed file with 18 additions and 4 deletions.
22 changes: 18 additions & 4 deletions crates/ide/src/inlay_hints/implicit_drop.rs
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,11 @@ pub(super) fn hints(
match_ast! {
match expr {
ast::BlockExpr(x) => x.stmt_list().and_then(|x| x.r_curly_token()).map(|x| x.text_range()).unwrap_or_else(|| expr.text_range()),
_ => expr.text_range(),
// make the inlay hint appear after the semicolon if there is
_ => {
let nearest_semicolon = nearest_token_after_node(expr, syntax::SyntaxKind::SEMICOLON);
nearest_semicolon.map(|x| x.text_range()).unwrap_or_else(|| expr.text_range())
},
}
}
}
Expand Down Expand Up @@ -95,7 +99,7 @@ pub(super) fn hints(
label.append_str(")");
acc.push(InlayHint {
range,
position: InlayHintPosition::Before,
position: InlayHintPosition::After,
pad_left: true,
pad_right: true,
kind: InlayKind::Drop,
Expand All @@ -109,6 +113,16 @@ pub(super) fn hints(
Some(())
}

fn nearest_token_after_node(
node: &syntax::SyntaxNode,
token_type: syntax::SyntaxKind,
) -> Option<syntax::SyntaxToken> {
node.siblings_with_tokens(syntax::Direction::Next)
.filter_map(|it| it.as_token().map(|it| it.clone()))
.filter(|it| it.kind() == token_type)
.next()
}

#[cfg(test)]
mod tests {
use crate::{
Expand All @@ -129,7 +143,7 @@ mod tests {
let x = X;
if 2 == 5 {
return;
//^^^^^^ drop(x)
//^ drop(x)
}
}
//^ drop(x)
Expand Down Expand Up @@ -176,7 +190,7 @@ mod tests {
let x = X;
let t_opt = Some(2);
let t = t_opt?;
//^^^^^^ drop(x)
//^ drop(x)
Some(())
}
//^ drop(x)
Expand Down

0 comments on commit bc9c952

Please sign in to comment.