minor touchups

jesse-shopify · jesse-shopify · commit decd5793ecac · 2026-01-08T13:45:13.000-08:00
diff --git a/rust/saturn/src/resolution.rs b/rust/saturn/src/resolution.rs
@@ -1055,12 +1055,9 @@ fn search_top_level(graph: &Graph, str_id: StringId) -> Outcome {
 }
 
 fn members_of(decl: &Declaration) -> &IdentityHashMap<StringId, DeclarationId> {
-    match decl {
-        Declaration::Class(it) => it.members(),
-        Declaration::SingletonClass(it) => it.members(),
-        Declaration::Module(it) => it.members(),
-        _ => panic!("Tried to get members for a declaration that isn't a namespace"),
-    }
+    decl.as_namespace()
+        .expect("Tried to get members for a declaration that isn't a namespace")
+        .members()
 }
 
 /// Returns a complexity score for a given name, which is used to sort names for resolution. The complexity is based

Original file line number	Diff line number	Diff line change
`@@ -1055,12 +1055,9 @@ fn search_top_level(graph: &Graph, str_id: StringId) -> Outcome {`
`1055`	`1055`	`}`
`1056`	`1056`
`1057`	`1057`	`fn members_of(decl: &Declaration) -> &IdentityHashMap<StringId, DeclarationId> {`
`1058`		`- match decl {`
`1059`		`- Declaration::Class(it) => it.members(),`
`1060`		`- Declaration::SingletonClass(it) => it.members(),`
`1061`		`- Declaration::Module(it) => it.members(),`
`1062`		`- _ => panic!("Tried to get members for a declaration that isn't a namespace"),`
`1063`		`- }`
	`1058`	`+ decl.as_namespace()`
	`1059`	`+ .expect("Tried to get members for a declaration that isn't a namespace")`
	`1060`	`+ .members()`
`1064`	`1061`	`}`
`1065`	`1062`
`1066`	`1063`	`/// Returns a complexity score for a given name, which is used to sort names for resolution. The complexity is based`