Skip to content

works - #1

Draft
nathanzeng wants to merge 1 commit into
masterfrom
dir-hidden
Draft

works#1
nathanzeng wants to merge 1 commit into
masterfrom
dir-hidden

Conversation

@nathanzeng

Copy link
Copy Markdown
Owner

No description provided.

nathanzeng pushed a commit that referenced this pull request Aug 18, 2026
Problem:
TSan CI, always at exit:

    WARNING: ThreadSanitizer: use of an invalid mutex (e.g. uninitialized or destroyed)
        #1 uv_mutex_lock
        #2 tui_flush src/nvim/tui/tui.c:1708
        neovim#3 ui_client_event_flush
        neovim#4 parse_msgpack src/nvim/msgpack_rpc/channel.c:259
        ...
        neovim#10 event_teardown src/nvim/main.c:187
        neovim#11 os_exit src/nvim/main.c:720
        ...
        neovim#19 tinput_done_event src/nvim/tui/input.c:184

Analysis (guess):
`rpc_close_event()` may run during `event_teardown()` and reattach to
a restarted server, then `ui_client_attached=true` gets set and redraws
resume in an already-stopped TUI.

Solution:
Check `!exiting` in `rpc_close_event`, like we already do in
`channel_proc_exit_cb()`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant