Skip to content

Commit de28baf

Browse files
committed
Auto merge of #15424 - lnicola:remove-remove-dbg-unwrap, r=lnicola
minor: Remove `unwrap` from `Remove dbg!` Part of #15398.
2 parents 1b67823 + b658f9a commit de28baf

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

crates/ide-assists/src/handlers/remove_dbg.rs

+1-4
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,11 @@ pub(crate) fn remove_dbg(acc: &mut Assists, ctx: &AssistContext<'_>) -> Option<(
3939

4040
let replacements =
4141
macro_calls.into_iter().filter_map(compute_dbg_replacement).collect::<Vec<_>>();
42-
if replacements.is_empty() {
43-
return None;
44-
}
4542

4643
acc.add(
4744
AssistId("remove_dbg", AssistKind::Refactor),
4845
"Remove dbg!()",
49-
replacements.iter().map(|&(range, _)| range).reduce(|acc, range| acc.cover(range)).unwrap(),
46+
replacements.iter().map(|&(range, _)| range).reduce(|acc, range| acc.cover(range))?,
5047
|builder| {
5148
for (range, expr) in replacements {
5249
if let Some(expr) = expr {

0 commit comments

Comments
 (0)