Add clarifying assertion

stepancheg · facebook-github-bot · commit 15021ff9765b · 2023-09-04T09:57:50.000-07:00
Reviewed By: IanChilds

Differential Revision: D48940697

fbshipit-source-id: 19ed17f500b02a6788aab28ac7df9f8cbe6c3cee
diff --git a/starlark/src/eval/compiler/scope/mod.rs b/starlark/src/eval/compiler/scope/mod.rs
@@ -814,7 +814,9 @@ impl<'f> ModuleScopeBuilder<'f> {
                 if self.locals.len() > 1 {
                     binding.captured = Captured::Yes;
                 }
-                Some((binding.resolved_slot(&self.codemap).unwrap(), binding_id))
+                let slot = binding.resolved_slot(&self.codemap).unwrap();
+                assert!(matches!(slot, Slot::Module(_)));
+                Some((slot, binding_id))
             }
             None => None,
         }

Original file line number	Diff line number	Diff line change
`@@ -814,7 +814,9 @@ impl<'f> ModuleScopeBuilder<'f> {`
`814`	`814`	`if self.locals.len() > 1 {`
`815`	`815`	`binding.captured = Captured::Yes;`
`816`	`816`	`}`
`817`		`- Some((binding.resolved_slot(&self.codemap).unwrap(), binding_id))`
	`817`	`+ let slot = binding.resolved_slot(&self.codemap).unwrap();`
	`818`	`+ assert!(matches!(slot, Slot::Module(_)));`
	`819`	`+ Some((slot, binding_id))`
`818`	`820`	`}`
`819`	`821`	`None => None,`
`820`	`822`	`}`