Remove PartialEq from DepNodeColor

oli-obk · oli-obk · commit 8bdb617f729f · 2024-06-05T15:46:36.000Z
diff --git a/compiler/rustc_query_system/src/dep_graph/graph.rs b/compiler/rustc_query_system/src/dep_graph/graph.rs
@@ -64,7 +64,6 @@ pub struct MarkFrame<'a> {
     parent: Option<&'a MarkFrame<'a>>,
 }
 
-#[derive(PartialEq)]
 enum DepNodeColor {
     Red,
     Green(DepNodeIndex),
@@ -925,7 +924,7 @@ impl<D: Deps> DepGraph<D> {
     /// Returns true if the given node has been marked as red during the
     /// current compilation session. Used in various assertions
     pub fn is_red(&self, dep_node: &DepNode) -> bool {
-        self.node_color(dep_node) == Some(DepNodeColor::Red)
+        matches!(self.node_color(dep_node), Some(DepNodeColor::Red))
     }
 
     /// Returns true if the given node has been marked as green during the

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,6 @@ pub struct MarkFrame<'a> {`
`64`	`64`	`parent: Option<&'a MarkFrame<'a>>,`
`65`	`65`	`}`
`66`	`66`
`67`		`-#[derive(PartialEq)]`
`68`	`67`	`enum DepNodeColor {`
`69`	`68`	`Red,`
`70`	`69`	`Green(DepNodeIndex),`
`@@ -925,7 +924,7 @@ impl<D: Deps> DepGraph<D> {`
`925`	`924`	`/// Returns true if the given node has been marked as red during the`
`926`	`925`	`/// current compilation session. Used in various assertions`
`927`	`926`	`pub fn is_red(&self, dep_node: &DepNode) -> bool {`
`928`		`- self.node_color(dep_node) == Some(DepNodeColor::Red)`
	`927`	`+ matches!(self.node_color(dep_node), Some(DepNodeColor::Red))`
`929`	`928`	`}`
`930`	`929`
`931`	`930`	`/// Returns true if the given node has been marked as green during the`