Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

invalid window id #337

Open
tropical32 opened this issue Oct 30, 2022 · 4 comments · May be fixed by #340
Open

invalid window id #337

tropical32 opened this issue Oct 30, 2022 · 4 comments · May be fixed by #340

Comments

@tropical32
Copy link

tropical32 commented Oct 30, 2022

This error is pretty hard to reproduce because it shows up inconsistently. I have two windows open in 1 tab and when I use the command HopChar2MW to jump to 2 characters I often get this error:

E5108: Error executing lua .../nvim/site/pack/packer/start/hop.nvim/lua/hop/window.lua:67: Invalid window id: 1012
stack traceback:
        [C]: in function 'nvim_win_get_buf'
        .../nvim/site/pack/packer/start/hop.nvim/lua/hop/window.lua:67: in function 'get_window_context'
        ...re/nvim/site/pack/packer/start/hop.nvim/lua/hop/init.lua:46: in function 'create_hint_state'
        ...re/nvim/site/pack/packer/start/hop.nvim/lua/hop/init.lua:328: in function 'hint_with_callback'
        ...re/nvim/site/pack/packer/start/hop.nvim/lua/hop/init.lua:310: in function 'hint_with'
        ...re/nvim/site/pack/packer/start/hop.nvim/lua/hop/init.lua:546: in function 'hint_char2'
        [string ":lua"]:1: in main chunk

Setup

lua require('hop').setup { keys = 'tnseriaodhfmluvcwyxzbpgjkq' }
nmap s :HopChar2MW<cr>

Version

NVIM v0.8.0
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by builduser

Features: +acl +iconv +tui
@hadronized
Copy link
Owner

Yeah I know about it. It’s because of virtual windows (popups, etc.). I think I already track that in another issue.

@zsmatrix62
Copy link

I encountered the problem today as well, cannot fix it by rolling back the commits ... hopping be resolved soon. Thanks.

@tropical32 tropical32 linked a pull request Nov 10, 2022 that will close this issue
@JulesNP
Copy link

JulesNP commented Nov 11, 2022

I'm experiencing the same error. I've had to turn off multiwindow until it gets fixed.

@honghao71
Copy link

I encountered the problem today as well, cannot fix it by rolling back the commits ... hopping be resolved soon. Thanks.

The fix #340 is working

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants