holm/nushell
1
1
Fork 0
forked from extern/nushell
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Do not set visibility to true automatically (#4591)

Browse Source 
Adding it by default grows the size of the visibility structure a lot.
This commit is contained in:
Jakub Žádník
2022-02-21 23:42:31 +02:00
committed by GitHub
parent 359bb6eebe
commit 739e403cd5
2 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/nu-protocol/src/engine/engine_state.rs
View File

@ -743,7 +743,6 @@ impl<'a> StateWorkingSet<'a> {
.expect("internal error: missing required scope frame");
scope_frame.decls.insert(name, decl_id);
scope_frame.visibility.use_decl_id(&decl_id);
decl_id
}
@ -790,7 +789,6 @@ impl<'a> StateWorkingSet<'a> {
if let Some(decl_id) = scope_frame.predecls.remove(name) {
scope_frame.decls.insert(name.into(), decl_id);
scope_frame.visibility.use_decl_id(&decl_id);
return Some(decl_id);
}