junegunn / limelight.vim

:flashlight: All the world's indeed a stage and we are merely players
MIT License
2.36k stars 54 forks source link

W19: Deleting augroup that is still in use #36

Closed lifepillar closed 8 years ago

lifepillar commented 8 years ago

I get the warning in the subject every time I leave Goyo. This is my configuration:

    set t_8f=^[[38;2;%lu;%lu;%lum
    set t_8b=^[[48;2;%lu;%lu;%lum
    set termguicolors

    fun! s:goyoEnter()
      Limelight
    endf

    fun! s:goyoLeave()
      Limelight!
    endf

    autocmd! User GoyoEnter nested call <sid>goyoEnter()
    autocmd! User GoyoLeave nested call <sid>goyoLeave()
lifepillar commented 8 years ago

To reproduce it with a minimal vimrc, you may need to call :colorscheme at least once (no idea why).

junegunn commented 8 years ago

Yeah, I also see the message. However I can't really understand why Vim is complaining when Limelight properly removes autocmds before trying to delete the group (see https://github.com/junegunn/limelight.vim/blob/master/autoload/limelight.vim#L240-L243). This could be a bug of Vim and I'm seeing the same problem in other plugins as well. I'll investigate when I get some time. Thanks.

lifepillar commented 8 years ago

It may be worth following this thread on vim-dev: https://groups.yahoo.com/neo/groups/vimdev/conversations/topics/93148

lifepillar commented 8 years ago

Eventually it was a Vim bug, fixed with https://github.com/vim/vim/releases/tag/v7.4.2300. You may close this.

junegunn commented 8 years ago

I see, thanks for the follow-up.