search

tommcdo / vim-fugitive-blame-ext

Extend tpope/vim-fugitive to show commit message on statusline in :Gblame
54 stars 2 forks source link

Consider caching commit messages #3

Closed mattboehm closed 11 years ago

mattboehm commented 11 years ago

When doing a Gblame, many lines may refer to the same commit. rather than shelling out to git every time, consider caching the messages in a dict locally.

Not sure if there's a significant performance change from doing this or not, but thought I'd throw it out there.

tommcdo commented 11 years ago

I thought I was doing this. I'm not totally sure if my cache is being shared across multiple :Gblames, but I think it is.

tommcdo commented 11 years ago

https://github.com/tommcdo/vim-fugitive-blame-ext/blob/master/plugin/fugitive-blame-ext.vim#L5

mattboehm commented 11 years ago

D'oh! how'd I miss that? Must be tired x.x.