vim/YouCompleteMe
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Auto merge of #2644 - micbou:improve-reparse-requirement, r=Valloric

Browse Source 
[READY] Improve reparse requirement on BufEnter event

When opening a file, Vim triggers the `BufRead` event then the `BufEnter` one. By setting the `s:previous_allowed_buffer_number` variable in `s:AllowedToCompleteInBuffer`, we avoid a reparse on `BufEnter` after parsing the buffer on `BufRead`. This reduces the number of `BufferVisit` and `FileReadyToParse` requests sent to the server by one when opening a file.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/youcompleteme/2644)
<!-- Reviewable:end -->
This commit is contained in:
zzbot 2017-05-15 12:06:51 -07:00 committed by GitHub
commit 99be8d0226
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
autoload/youcompleteme.vim
View File

@ -345,7 +345,11 @@ function! s:AllowedToCompleteInBuffer( buffer )
\ has_key( g:ycm_filetype_whitelist, buffer_filetype )
let blacklist_allows = !has_key( g:ycm_filetype_blacklist, buffer_filetype )
return whitelist_allows && blacklist_allows
let allowed = whitelist_allows && blacklist_allows
if allowed
let s:previous_allowed_buffer_number = bufnr( a:buffer )
endif
return allowed
endfunction
@ -355,15 +359,11 @@ endfunction
function! s:VisitedBufferRequiresReparse()
if !s:AllowedToCompleteInCurrentBuffer()
if bufnr( '%' ) ==# s:previous_allowed_buffer_number
return 0
endif
if bufnr( '' ) ==# s:previous_allowed_buffer_number
return 0
endif
let s:previous_allowed_buffer_number = bufnr( '' )
return 1
return s:AllowedToCompleteInCurrentBuffer()
endfunction