extern/nushell
1
0
Fork 1
mirror of https://github.com/nushell/nushell.git synced 2025-05-31 23:26:11 +02:00
Code Issues Packages Projects Releases Wiki Activity

doc: todo comment about variable shadowing

Browse Source
This commit is contained in:
blindfs 2025-04-08 21:18:26 +08:00
parent aff75c7d93
commit 7915e14463
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
crates/nu-lsp/src/ast.rs
View File

@ -161,8 +161,10 @@ fn try_find_id_in_use(
.get(name) .get(name)
.cloned() .cloned()
.or_else(|| { .or_else(|| {
// NOTE: for module record variable: https://www.nushell.sh/book/modules/using_modules.html#importing-constants // TODO: This is for the module record variable:
// the definition span is located at the head of the use command // https://www.nushell.sh/book/modules/using_modules.html#importing-constants
// The definition span is located at the head of the `use` command.
// This method won't work if the variable is later shadowed by a new definition.
let var_id = working_set.find_variable(name)?; let var_id = working_set.find_variable(name)?;
call.span() call.span()
.contains_span(working_set.get_variable(var_id).declaration_span) .contains_span(working_set.get_variable(var_id).declaration_span)