Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit 340bd2a

Browse files
nyuriknyurik
committed
Change a few &Option<T> into Option<&T> 
1 parent b6576e3 commit 340bd2a

File tree

32 files changed

+144
-116
lines changed

32 files changed

+144
-116
lines changed

‎compiler/rustc_ast_lowering/src/asm.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
198198
if let Some(def_id) = static_def_id {
199199
let path = self.lower_qpath(
200200
sym.id,
201-
&sym.qself,
201+
sym.qself.as_ref(),
202202
&sym.path,
203203
ParamMode::Optional,
204204
AllowReturnTypeNotation::No,

‎compiler/rustc_ast_lowering/src/delegation.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -337,7 +337,7 @@ impl<'hir> LoweringContext<'_, 'hir> {
337337
} else {
338338
let path = self.lower_qpath(
339339
delegation.id,
340-
&delegation.qself,
340+
delegation.qself.as_ref(),
341341
&delegation.path,
342342
ParamMode::Optional,
343343
AllowReturnTypeNotation::No,

‎compiler/rustc_ast_lowering/src/expr.rs‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -278,7 +278,7 @@ impl<'hir> LoweringContext<'_, 'hir> {
278278
ExprKind::Path(qself, path) => {
279279
let qpath = self.lower_qpath(
280280
e.id,
281-
qself,
281+
qself.as_ref(),
282282
path,
283283
ParamMode::Optional,
284284
AllowReturnTypeNotation::No,
@@ -326,7 +326,7 @@ impl<'hir> LoweringContext<'_, 'hir> {
326326
hir::ExprKind::Struct(
327327
self.arena.alloc(self.lower_qpath(
328328
e.id,
329-
&se.qself,
329+
se.qself.as_ref(),
330330
&se.path,
331331
ParamMode::Optional,
332332
AllowReturnTypeNotation::No,
@@ -1287,7 +1287,7 @@ impl<'hir> LoweringContext<'_, 'hir> {
12871287
);
12881288
let qpath = self.lower_qpath(
12891289
callee.id,
1290-
qself,
1290+
qself.as_ref(),
12911291
path,
12921292
ParamMode::Optional,
12931293
AllowReturnTypeNotation::No,
@@ -1308,7 +1308,7 @@ impl<'hir> LoweringContext<'_, 'hir> {
13081308
if let Some((qself, path)) = self.extract_unit_struct_path(lhs) {
13091309
let qpath = self.lower_qpath(
13101310
lhs.id,
1311-
qself,
1311+
qself.as_ref(),
13121312
path,
13131313
ParamMode::Optional,
13141314
AllowReturnTypeNotation::No,
@@ -1334,7 +1334,7 @@ impl<'hir> LoweringContext<'_, 'hir> {
13341334
}));
13351335
let qpath = self.lower_qpath(
13361336
lhs.id,
1337-
&se.qself,
1337+
se.qself.as_ref(),
13381338
&se.path,
13391339
ParamMode::Optional,
13401340
AllowReturnTypeNotation::No,

‎compiler/rustc_ast_lowering/src/format.rs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ fn make_argument<'hir>(
283283
fn make_count<'hir>(
284284
ctx: &mut LoweringContext<'_, 'hir>,
285285
sp: Span,
286-
count: &Option<FormatCount>,
286+
count: Option<&FormatCount>,
287287
argmap: &mut FxIndexMap<(usize, ArgumentType), Option<Span>>,
288288
) -> hir::Expr<'hir> {
289289
match count {
@@ -378,8 +378,8 @@ fn make_format_spec<'hir>(
378378
| ((debug_hex == Some(FormatDebugHex::Lower)) as u32) << 4
379379
| ((debug_hex == Some(FormatDebugHex::Upper)) as u32) << 5;
380380
let flags = ctx.expr_u32(sp, flags);
381-
let precision = make_count(ctx, sp, precision, argmap);
382-
let width = make_count(ctx, sp, width, argmap);
381+
let precision = make_count(ctx, sp, precision.as_ref(), argmap);
382+
let width = make_count(ctx, sp, width.as_ref(), argmap);
383383
let format_placeholder_new = ctx.arena.alloc(ctx.expr_lang_item_type_relative(
384384
sp,
385385
hir::LangItem::FormatPlaceholder,

‎compiler/rustc_ast_lowering/src/lib.rs‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1206,7 +1206,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
12061206
fn lower_path_ty(
12071207
&mut self,
12081208
t: &Ty,
1209-
qself: &Option<ptr::P<QSelf>>,
1209+
qself: Option<&ptr::P<QSelf>>,
12101210
path: &Path,
12111211
param_mode: ParamMode,
12121212
itctx: ImplTraitContext,
@@ -1340,7 +1340,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
13401340
return self.lower_ty_direct(ty, itctx);
13411341
}
13421342
TyKind::Path(qself, path) => {
1343-
return self.lower_path_ty(t, qself, path, ParamMode::Explicit, itctx);
1343+
return self.lower_path_ty(t, qself.as_ref(), path, ParamMode::Explicit, itctx);
13441344
}
13451345
TyKind::ImplicitSelf => {
13461346
let hir_id = self.next_id();
@@ -2222,7 +2222,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
22222222
) -> hir::TraitRef<'hir> {
22232223
let path = match self.lower_qpath(
22242224
p.ref_id,
2225-
&None,
2225+
None,
22262226
&p.path,
22272227
ParamMode::Explicit,
22282228
AllowReturnTypeNotation::No,
@@ -2361,7 +2361,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
23612361
Res::Def(DefKind::ConstParam, _) => {
23622362
let qpath = self.lower_qpath(
23632363
ty_id,
2364-
&None,
2364+
None,
23652365
path,
23662366
ParamMode::Optional,
23672367
AllowReturnTypeNotation::No,
@@ -2440,7 +2440,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
24402440
{
24412441
let qpath = self.lower_qpath(
24422442
expr.id,
2443-
qself,
2443+
qself.as_ref(),
24442444
path,
24452445
ParamMode::Optional,
24462446
AllowReturnTypeNotation::No,

‎compiler/rustc_ast_lowering/src/pat.rs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
3535
PatKind::TupleStruct(qself, path, pats) => {
3636
let qpath = self.lower_qpath(
3737
pattern.id,
38-
qself,
38+
qself.as_ref(),
3939
path,
4040
ParamMode::Optional,
4141
AllowReturnTypeNotation::No,
@@ -53,7 +53,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
5353
PatKind::Path(qself, path) => {
5454
let qpath = self.lower_qpath(
5555
pattern.id,
56-
qself,
56+
qself.as_ref(),
5757
path,
5858
ParamMode::Optional,
5959
AllowReturnTypeNotation::No,
@@ -65,7 +65,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
6565
PatKind::Struct(qself, path, fields, etc) => {
6666
let qpath = self.lower_qpath(
6767
pattern.id,
68-
qself,
68+
qself.as_ref(),
6969
path,
7070
ParamMode::Optional,
7171
AllowReturnTypeNotation::No,

‎compiler/rustc_ast_lowering/src/path.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ impl<'a, 'hir> LoweringContext<'a, 'hir> {
2525
pub(crate) fn lower_qpath(
2626
&mut self,
2727
id: NodeId,
28-
qself: &Option<ptr::P<QSelf>>,
28+
qself: Option<&ptr::P<QSelf>>,
2929
p: &Path,
3030
param_mode: ParamMode,
3131
allow_return_type_notation: AllowReturnTypeNotation,

‎compiler/rustc_ast_pretty/src/pprust/state.rs‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1107,9 +1107,9 @@ impl<'a> State<'a> {
11071107
self.commasep_cmnt(b, exprs, |s, e| s.print_expr(e, FixupContext::default()), |e| e.span)
11081108
}
11091109

1110-
pub fn print_opt_lifetime(&mut self, lifetime: &Option<ast::Lifetime>) {
1111-
if let Some(lt) = *lifetime {
1112-
self.print_lifetime(lt);
1110+
pub fn print_opt_lifetime(&mut self, lifetime: Option<&ast::Lifetime>) {
1111+
if let Some(lt) = lifetime {
1112+
self.print_lifetime(*lt);
11131113
self.nbsp();
11141114
}
11151115
}
@@ -1160,7 +1160,7 @@ impl<'a> State<'a> {
11601160
}
11611161
ast::TyKind::Ref(lifetime, mt) => {
11621162
self.word("&");
1163-
self.print_opt_lifetime(lifetime);
1163+
self.print_opt_lifetime(lifetime.as_ref());
11641164
self.print_mt(mt, false);
11651165
}
11661166
ast::TyKind::Never => {
@@ -1709,7 +1709,7 @@ impl<'a> State<'a> {
17091709
}
17101710
SelfKind::Region(lt, m) => {
17111711
self.word("&");
1712-
self.print_opt_lifetime(lt);
1712+
self.print_opt_lifetime(lt.as_ref());
17131713
self.print_mutability(*m, false);
17141714
self.word("self")
17151715
}

‎compiler/rustc_ast_pretty/src/pprust/state/expr.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ impl<'a> State<'a> {
151151

152152
fn print_expr_struct(
153153
&mut self,
154-
qself: &Option<P<ast::QSelf>>,
154+
qself: Option<&P<ast::QSelf>>,
155155
path: &ast::Path,
156156
fields: &[ast::ExprField],
157157
rest: &ast::StructRest,
@@ -388,7 +388,7 @@ impl<'a> State<'a> {
388388
self.print_expr_repeat(element, count);
389389
}
390390
ast::ExprKind::Struct(se) => {
391-
self.print_expr_struct(&se.qself, &se.path, &se.fields, &se.rest);
391+
self.print_expr_struct(se.qself.as_ref(), &se.path, &se.fields, &se.rest);
392392
}
393393
ast::ExprKind::Tup(exprs) => {
394394
self.print_expr_tup(exprs);

‎compiler/rustc_ast_pretty/src/pprust/state/item.rs‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -386,18 +386,18 @@ impl<'a> State<'a> {
386386
ast::ItemKind::Delegation(deleg) => self.print_delegation(
387387
&item.attrs,
388388
&item.vis,
389-
&deleg.qself,
389+
deleg.qself.as_ref(),
390390
&deleg.path,
391391
DelegationKind::Single,
392-
&deleg.body,
392+
deleg.body.as_ref(),
393393
),
394394
ast::ItemKind::DelegationMac(deleg) => self.print_delegation(
395395
&item.attrs,
396396
&item.vis,
397-
&deleg.qself,
397+
deleg.qself.as_ref(),
398398
&deleg.prefix,
399399
deleg.suffixes.as_ref().map_or(DelegationKind::Glob, |s| DelegationKind::List(s)),
400-
&deleg.body,
400+
deleg.body.as_ref(),
401401
),
402402
}
403403
self.ann.post(self, AnnNode::Item(item))
@@ -578,18 +578,18 @@ impl<'a> State<'a> {
578578
ast::AssocItemKind::Delegation(deleg) => self.print_delegation(
579579
&item.attrs,
580580
vis,
581-
&deleg.qself,
581+
deleg.qself.as_ref(),
582582
&deleg.path,
583583
DelegationKind::Single,
584-
&deleg.body,
584+
deleg.body.as_ref(),
585585
),
586586
ast::AssocItemKind::DelegationMac(deleg) => self.print_delegation(
587587
&item.attrs,
588588
vis,
589-
&deleg.qself,
589+
deleg.qself.as_ref(),
590590
&deleg.prefix,
591591
deleg.suffixes.as_ref().map_or(DelegationKind::Glob, |s| DelegationKind::List(s)),
592-
&deleg.body,
592+
deleg.body.as_ref(),
593593
),
594594
}
595595
self.ann.post(self, AnnNode::SubItem(id))
@@ -599,10 +599,10 @@ impl<'a> State<'a> {
599599
&mut self,
600600
attrs: &[ast::Attribute],
601601
vis: &ast::Visibility,
602-
qself: &Option<P<ast::QSelf>>,
602+
qself: Option<&P<ast::QSelf>>,
603603
path: &ast::Path,
604604
kind: DelegationKind<'_>,
605-
body: &Option<P<ast::Block>>,
605+
body: Option<&P<ast::Block>>,
606606
) {
607607
if body.is_some() {
608608
self.head("");

0 commit comments

Comments
 (0)