Skip to content

Commit fd6b537

Browse files
committed
Auto merge of #79284 - Nadrieril:constructor-module, r=varkor
Split match exhaustiveness into two files I feel the constructor-related things in the `_match` module make enough sense on their own so I split them off. It makes `_match` feel less like a complicated mess. I'm not aware of PRs in progress against this module apart from my own so hopefully I'm not annoying too many people. I have a lot of questions about the conventions in naming and modules around the compiler. Like, why is the module named `_match`? Could I rename it to `usefulness` maybe? Should `deconstruct_pat` be a submodule of `_match` since only `_match` uses it? Is it ok to move big piles of code around even if it makes git blame more difficult? r? `@varkor` `@rustbot` modify labels: +A-exhaustiveness-checking
2 parents 6a88957 + 941c6ac commit fd6b537

File tree

4 files changed

+1737
-1712
lines changed

4 files changed

+1737
-1712
lines changed

compiler/rustc_mir_build/src/thir/pattern/check_match.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
use super::_match::Usefulness::*;
2-
use super::_match::{
1+
use super::usefulness::Usefulness::*;
2+
use super::usefulness::{
33
compute_match_usefulness, expand_pattern, MatchArm, MatchCheckCtxt, UsefulnessReport,
44
};
55
use super::{PatCtxt, PatKind, PatternError};

0 commit comments

Comments
 (0)