Skip to content

Commit 94fb87a

Browse files
committed
Remove duplicate impl of string unescape
1 parent 2af87ea commit 94fb87a

File tree

5 files changed

+447
-547
lines changed

5 files changed

+447
-547
lines changed

compiler/rustc_builtin_macros/src/format.rs

+13-6
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
use std::ops::Range;
2+
13
use parse::Position::ArgumentNamed;
24
use rustc_ast::ptr::P;
35
use rustc_ast::tokenstream::TokenStream;
@@ -334,7 +336,7 @@ fn make_format_args(
334336
return ExpandResult::Ready(Err(guar));
335337
}
336338

337-
let to_span = |inner_span: parse::InnerSpan| {
339+
let to_span = |inner_span: Range<usize>| {
338340
is_source_literal.then(|| {
339341
fmt_span.from_inner(InnerSpan { start: inner_span.start, end: inner_span.end })
340342
})
@@ -406,7 +408,7 @@ fn make_format_args(
406408
let mut placeholder_index = 0;
407409

408410
for piece in &pieces {
409-
match *piece {
411+
match piece.clone() {
410412
parse::Piece::Lit(s) => {
411413
unfinished_literal.push_str(s);
412414
}
@@ -416,7 +418,8 @@ fn make_format_args(
416418
unfinished_literal.clear();
417419
}
418420

419-
let span = parser.arg_places.get(placeholder_index).and_then(|&s| to_span(s));
421+
let span =
422+
parser.arg_places.get(placeholder_index).and_then(|s| to_span(s.clone()));
420423
placeholder_index += 1;
421424

422425
let position_span = to_span(position_span);
@@ -608,7 +611,7 @@ fn make_format_args(
608611
fn invalid_placeholder_type_error(
609612
ecx: &ExtCtxt<'_>,
610613
ty: &str,
611-
ty_span: Option<parse::InnerSpan>,
614+
ty_span: Option<Range<usize>>,
612615
fmt_span: Span,
613616
) {
614617
let sp = ty_span.map(|sp| fmt_span.from_inner(InnerSpan::new(sp.start, sp.end)));
@@ -774,8 +777,12 @@ fn report_redundant_format_arguments<'a>(
774777
let mut fmt_spans = vec![];
775778

776779
for (i, unnamed_arg) in args.unnamed_args().iter().enumerate().rev() {
777-
let Some(ty) = unnamed_arg.expr.to_ty() else { continue };
778-
let Some(argument_binding) = ty.kind.is_simple_path() else { continue };
780+
let Some(ty) = unnamed_arg.expr.to_ty() else {
781+
continue;
782+
};
783+
let Some(argument_binding) = ty.kind.is_simple_path() else {
784+
continue;
785+
};
779786
let argument_binding = argument_binding.as_str();
780787

781788
if used[i] {

0 commit comments

Comments
 (0)