-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Invalid lnum when using Fugitive Glog command #16
Comments
Note that my patch merely silences the error message, but doesn't do anything about the cause. However, I think in the case of this particular interaction the root cause isn't a bug, just different expectations about what the quickfix list will contain. The quickfixsigns.vim code expects the quickfix list to be populated by things like compilation errors, which come with line numbers -- and if those line numbers are missing, it's a bug. But when the quickfix list is being populated by fugitive via the :Glog command (which runs git log), it's populating the quickfix list from the Git log, which means that the absence of line numbers is expected. So depending on what is populating the quickfix list, the absence of line numbers may or may not be a bug, which means that my quick-and-dirty hack of a patch, which just silences the error message, will not be the right solution in every case. But it's certainly enough to remove the pain of using Fugitive and quickfixsigns.vim together, which is all I cared about earlier today when I was trying to look at some git logs. So take this patch with a grain of salt. :-) |
But why does fugitive populate the quickfixlist in the first place? |
Thanks for the quick fix @tomtom! |
When using Fugitive in conjunction with quickfixsigns I'm seeing the following error print out for every line in the file when running
:Glog
@rmunn tracked down the error and reported it on my dot_vim repository. Here's what he said could fix the problem:
The text was updated successfully, but these errors were encountered: