Skip to content

Commit 422856d

Browse files
bors[bot]matklad
andauthored
Merge #5812
5812: Minor cleanups r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
2 parents 63ac896 + a3b0a3a commit 422856d

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

crates/hir/src/semantics.rs

+6-7
Original file line numberDiff line numberDiff line change
@@ -613,14 +613,13 @@ impl<'db> SemanticsImpl<'db> {
613613
InFile::new(file_id, node)
614614
}
615615

616-
pub fn is_unsafe_method_call(&self, method_call_expr: &ast::MethodCallExpr) -> bool {
616+
fn is_unsafe_method_call(&self, method_call_expr: &ast::MethodCallExpr) -> bool {
617617
method_call_expr
618618
.expr()
619619
.and_then(|expr| {
620-
let field_expr = if let ast::Expr::FieldExpr(field_expr) = expr {
621-
field_expr
622-
} else {
623-
return None;
620+
let field_expr = match expr {
621+
ast::Expr::FieldExpr(field_expr) => field_expr,
622+
_ => return None,
624623
};
625624
let ty = self.type_of_expr(&field_expr.expr()?)?;
626625
if !ty.is_packed(self.db) {
@@ -635,7 +634,7 @@ impl<'db> SemanticsImpl<'db> {
635634
.unwrap_or(false)
636635
}
637636

638-
pub fn is_unsafe_ref_expr(&self, ref_expr: &ast::RefExpr) -> bool {
637+
fn is_unsafe_ref_expr(&self, ref_expr: &ast::RefExpr) -> bool {
639638
ref_expr
640639
.expr()
641640
.and_then(|expr| {
@@ -654,7 +653,7 @@ impl<'db> SemanticsImpl<'db> {
654653
// more than it should with the current implementation.
655654
}
656655

657-
pub fn is_unsafe_ident_pat(&self, ident_pat: &ast::IdentPat) -> bool {
656+
fn is_unsafe_ident_pat(&self, ident_pat: &ast::IdentPat) -> bool {
658657
if !ident_pat.ref_token().is_some() {
659658
return false;
660659
}

0 commit comments

Comments
 (0)