Skip to content

Commit 12df480

Browse files
Apply suggestions from code review
Co-authored-by: Jeong YunWon <youknowone@users.noreply.github.com>
1 parent 96a0ea9 commit 12df480

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

compiler/src/symboltable.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -473,11 +473,12 @@ impl SymbolTableBuilder {
473473
}
474474

475475
fn scan_parameter(&mut self, parameter: &ast::Parameter) -> SymbolTableResult {
476-
if let Some(_annotation) = &parameter.annotation {
477-
self.register_name(&parameter.arg, SymbolUsage::AnnotationParameter)
476+
let usage = if parameter.annotation.is_some() {
477+
SymbolUsage::AnnotationParamater
478478
} else {
479-
self.register_name(&parameter.arg, SymbolUsage::Parameter)
480-
}
479+
SymbolUsage::Parameter
480+
};
481+
self.regsiter_name(&parameter.arg, usage)
481482
}
482483

483484
fn scan_parameters_annotations(&mut self, parameters: &[ast::Parameter]) -> SymbolTableResult {

vm/src/stdlib/symtable.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@ impl PySymbol {
249249

250250
#[pymethod(name = "is_namespace")]
251251
fn is_namespace(&self) -> bool {
252-
self.namespaces.len() > 0
252+
self.namespaces.is_empty()
253253
}
254254

255255
#[pymethod(name = "is_annotated")]

0 commit comments

Comments
 (0)