Public/nushell
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix delta not being merged when evaluating menus

Browse Source
This commit is contained in:
kubouch 2024-06-10 19:08:48 +03:00
parent be8c1dc006
commit addfb88dd6
2 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
crates/nu-cli/src/reedline_config.rs
View File

@ -76,7 +76,7 @@ const DEFAULT_HELP_MENU: &str = r#"
// Adds all menus to line editor
pub(crate) fn add_menus(
mut line_editor: Reedline,
engine_state: Arc<EngineState>,
engine_state: &mut Arc<EngineState>,
stack: &Stack,
config: &Config,
) -> Result<Reedline, ShellError> {
@ -100,8 +100,8 @@ pub(crate) fn add_menus(
.iter()
.any(|menu| menu.name.to_expanded_string("", config) == name)
{
let (block, _) = {
let mut working_set = StateWorkingSet::new(&engine_state);
let (block, delta) = {
let mut working_set = StateWorkingSet::new(engine_state);
let output = parse(
&mut working_set,
Some(name), // format!("entry #{}", entry_num)
@ -112,9 +112,13 @@ pub(crate) fn add_menus(
(output, working_set.render())
};
let _ = Arc::get_mut(engine_state)
.expect("no engine state")
.merge_delta(delta);
let mut temp_stack = Stack::new().capture();
let input = PipelineData::Empty;
let res = eval_block::<WithoutDebug>(&engine_state, &mut temp_stack, &block, input)?;
let res = eval_block::<WithoutDebug>(engine_state, &mut temp_stack, &block, input)?;
if let PipelineData::Value(value, None) = res {
for menu in create_menus(&value)? {

4
crates/nu-cli/src/repl.rs
View File

@ -345,7 +345,7 @@ fn loop_iteration(ctx: LoopContext) -> (bool, Stack, Reedline) {
use_color,
);
let engine_reference = Arc::new(engine_state.clone());
let mut engine_reference = Arc::new(engine_state.clone());
let config = engine_state.get_config();
start_time = std::time::Instant::now();
@ -428,7 +428,7 @@ fn loop_iteration(ctx: LoopContext) -> (bool, Stack, Reedline) {
start_time = std::time::Instant::now();
trace!("adding menus");
line_editor =
add_menus(line_editor, engine_reference, &stack_arc, config).unwrap_or_else(|e| {
add_menus(line_editor, &mut engine_reference, &stack_arc, config).unwrap_or_else(|e| {
report_error_new(engine_state, &e);
Reedline::create()
});