search

romainl / vim-qf

Tame the quickfix window.
MIT License
629 stars 34 forks source link

Fix issue #76 #115

Closed lacygoill closed 3 years ago

lacygoill commented 3 years ago

Improve the logic distinguishing the location list window from the quickfix list window by inspecting the 'loclist' key in the output of "getwininfo()".

When invoking "qf#SetList()" to clear the location list, bail out right before ":lclose|lwindow". Rationale: ":lwindow" will necessarily fail, causing the subsequent invocation of "qf#SetList()" to unconditionally use the quickfix list window. We don't want that. We want to stay in the location list window if we're already in there.

romainl commented 3 years ago

Thank you @lacygoill.

By the way, I couldn't help but notice your highly annotated vimrc and all those vim9 rewrites. Would you mind shedding some light on what this is all about?

lacygoill commented 3 years ago

By the way, I couldn't help but notice your highly annotated vimrc and all those vim9 rewrites. Would you mind shedding some light on what this is all about?

I don't know. People refactor code in rust, in lua, ... why not in Vim9? As for the rewrites, it's easier to fix bugs in my plugins than in other people's plugins. They might be too busy to fix them, or their code might look too weird for me.

romainl commented 3 years ago

That makes sense but it represents quite a lot of work. Kudos.