Skip to content

Commit fc05ee0

Browse files
committed
Rename
1 parent 6433f90 commit fc05ee0

File tree

1 file changed

+20
-19
lines changed
  • compiler/rustc_mir_build/src/build/matches

1 file changed

+20
-19
lines changed

compiler/rustc_mir_build/src/build/matches/util.rs

+20-19
Original file line numberDiff line numberDiff line change
@@ -95,27 +95,28 @@ impl<'a, 'tcx> Builder<'a, 'tcx> {
9595

9696
impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
9797
pub(in crate::build) fn new(
98-
mut place: PlaceBuilder<'tcx>,
98+
mut place_builder: PlaceBuilder<'tcx>,
9999
pattern: &'pat Pat<'tcx>,
100100
cx: &mut Builder<'_, 'tcx>,
101101
) -> MatchPair<'pat, 'tcx> {
102102
// Force the place type to the pattern's type.
103103
// FIXME(oli-obk): can we use this to simplify slice/array pattern hacks?
104-
if let Some(resolved) = place.resolve_upvar(cx) {
105-
place = resolved;
104+
if let Some(resolved) = place_builder.resolve_upvar(cx) {
105+
place_builder = resolved;
106106
}
107107

108108
// Only add the OpaqueCast projection if the given place is an opaque type and the
109109
// expected type from the pattern is not.
110-
let may_need_cast = match place.base() {
110+
let may_need_cast = match place_builder.base() {
111111
PlaceBase::Local(local) => {
112-
let ty = Place::ty_from(local, place.projection(), &cx.local_decls, cx.tcx).ty;
112+
let ty =
113+
Place::ty_from(local, place_builder.projection(), &cx.local_decls, cx.tcx).ty;
113114
ty != pattern.ty && ty.has_opaque_types()
114115
}
115116
_ => true,
116117
};
117118
if may_need_cast {
118-
place = place.project(ProjectionElem::OpaqueCast(pattern.ty));
119+
place_builder = place_builder.project(ProjectionElem::OpaqueCast(pattern.ty));
119120
}
120121

121122
let default_irrefutable = || TestCase::Irrefutable { binding: None, ascription: None };
@@ -124,7 +125,7 @@ impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
124125
PatKind::Never | PatKind::Wild | PatKind::Error(_) => default_irrefutable(),
125126
PatKind::Or { ref pats } => {
126127
let pats: Box<[_]> =
127-
pats.iter().map(|pat| FlatPat::new(place.clone(), pat, cx)).collect();
128+
pats.iter().map(|pat| FlatPat::new(place_builder.clone(), pat, cx)).collect();
128129
let simple = pats.iter().all(|fpat| fpat.simple);
129130
TestCase::Or { pats, simple }
130131
}
@@ -145,13 +146,13 @@ impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
145146
..
146147
} => {
147148
// Apply the type ascription to the value at `match_pair.place`
148-
let ascription = place.try_to_place(cx).map(|source| super::Ascription {
149+
let ascription = place_builder.try_to_place(cx).map(|source| super::Ascription {
149150
annotation: annotation.clone(),
150151
source,
151152
variance,
152153
});
153154

154-
subpairs.push(MatchPair::new(place.clone(), subpattern, cx));
155+
subpairs.push(MatchPair::new(place_builder.clone(), subpattern, cx));
155156
TestCase::Irrefutable { ascription, binding: None }
156157
}
157158

@@ -164,7 +165,7 @@ impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
164165
ref subpattern,
165166
is_primary: _,
166167
} => {
167-
let binding = place.try_to_place(cx).map(|source| super::Binding {
168+
let binding = place_builder.try_to_place(cx).map(|source| super::Binding {
168169
span: pattern.span,
169170
source,
170171
var_id: var,
@@ -173,14 +174,14 @@ impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
173174

174175
if let Some(subpattern) = subpattern.as_ref() {
175176
// this is the `x @ P` case; have to keep matching against `P` now
176-
subpairs.push(MatchPair::new(place.clone(), subpattern, cx));
177+
subpairs.push(MatchPair::new(place_builder.clone(), subpattern, cx));
177178
}
178179
TestCase::Irrefutable { ascription: None, binding }
179180
}
180181

181182
PatKind::InlineConstant { subpattern: ref pattern, def, .. } => {
182183
// Apply a type ascription for the inline constant to the value at `match_pair.place`
183-
let ascription = place.try_to_place(cx).map(|source| {
184+
let ascription = place_builder.try_to_place(cx).map(|source| {
184185
let span = pattern.span;
185186
let parent_id = cx.tcx.typeck_root_def_id(cx.def_id.to_def_id());
186187
let args = ty::InlineConstArgs::new(
@@ -206,16 +207,16 @@ impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
206207
super::Ascription { annotation, source, variance: ty::Contravariant }
207208
});
208209

209-
subpairs.push(MatchPair::new(place.clone(), pattern, cx));
210+
subpairs.push(MatchPair::new(place_builder.clone(), pattern, cx));
210211
TestCase::Irrefutable { ascription, binding: None }
211212
}
212213

213214
PatKind::Array { ref prefix, ref slice, ref suffix } => {
214-
cx.prefix_slice_suffix(&mut subpairs, &place, prefix, slice, suffix);
215+
cx.prefix_slice_suffix(&mut subpairs, &place_builder, prefix, slice, suffix);
215216
default_irrefutable()
216217
}
217218
PatKind::Slice { ref prefix, ref slice, ref suffix } => {
218-
cx.prefix_slice_suffix(&mut subpairs, &place, prefix, slice, suffix);
219+
cx.prefix_slice_suffix(&mut subpairs, &place_builder, prefix, slice, suffix);
219220

220221
if prefix.is_empty() && slice.is_some() && suffix.is_empty() {
221222
default_irrefutable()
@@ -228,7 +229,7 @@ impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
228229
}
229230

230231
PatKind::Variant { adt_def, variant_index, args, ref subpatterns } => {
231-
let downcast_place = place.clone().downcast(adt_def, variant_index); // `(x as Variant)`
232+
let downcast_place = place_builder.clone().downcast(adt_def, variant_index); // `(x as Variant)`
232233
subpairs = cx.field_match_pairs(downcast_place, subpatterns);
233234

234235
let irrefutable = adt_def.variants().iter_enumerated().all(|(i, v)| {
@@ -250,18 +251,18 @@ impl<'pat, 'tcx> MatchPair<'pat, 'tcx> {
250251
}
251252

252253
PatKind::Leaf { ref subpatterns } => {
253-
subpairs = cx.field_match_pairs(place.clone(), subpatterns);
254+
subpairs = cx.field_match_pairs(place_builder.clone(), subpatterns);
254255
default_irrefutable()
255256
}
256257

257258
PatKind::Deref { ref subpattern } => {
258-
let place_builder = place.clone().deref();
259+
let place_builder = place_builder.clone().deref();
259260
subpairs.push(MatchPair::new(place_builder, subpattern, cx));
260261
default_irrefutable()
261262
}
262263
};
263264

264-
MatchPair { place, test_case, subpairs, pattern }
265+
MatchPair { place: place_builder, test_case, subpairs, pattern }
265266
}
266267

267268
/// Whether this recursively contains no bindings or ascriptions.

0 commit comments

Comments
 (0)