call TextEditOutpu.state.store(...) causes the app to freeze

[lost22git]

 ui.ctx().input_mut(|i| {
            // Ctrl+J newline and auto-indent
            if cursor_range.is_some()
                && i.consume_shortcut(&KeyboardShortcut {
                    modifiers: Modifiers::CTRL,
                    key: Key::J,
                },)
            {
                let cr = cursor_range.unwrap();
                let row = cr.primary.rcursor.row;
                let line = f.code.lines().nth(row,).unwrap();
                tracing::info!("line => {line}");
                let mut space_count = 0;
                for c in line.chars() {
                    if c == ' ' {
                        space_count += 1;
                    } else if c == '\t' {
                        space_count += 4;
                    } else {
                        break;
                    }
                }
                let line = line.trim_end();
                if line.ends_with('{',) || line.ends_with('[',) || line.ends_with('(',) {
                    space_count += 4;
                }
                tracing::info!("space_count => {space_count}");
                let spaces = " ".repeat(space_count,);
                f.code.insert_text(
                    &format!("\n{spaces}"),
                    galley.cursor_end_of_row(&cr.primary,).ccursor.index,
                );
                editor_out.state.set_cursor_range(Some(CursorRange::one(
                    galley.cursor_end_of_row(&galley.cursor_down_one_row(&cr.primary,),),
                ),),);
                editor_out.state.store(ui.ctx(), code_editor_id,); // *** freeze app
                tracing::info!("==> store successfully!");
            }

[lost22git]

os => win11
eframe => 0.21.3

[YgorSouza]

I think this is the same problem as #2752, and if so, the solution in #2753 also applies. ui.ctx().input_mut acquires an exclusive lock to the context for the duration of the closure, and editor_out.state.store(ui.ctx(), ... also tries to acquire the same exclusive lock to modify the context, so it causes a deadlock. You have to move this part of the code outside the closure.

[lost22git]

@YgorSouza you are right. thanks! Hope to remind users on the doc ;-)