Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@daisuzu
Copy link

@daisuzu daisuzu commented Mar 25, 2017

Since 8.0.0050, an extra waiting occurs when reading completion candidates from other files in ins-completion.

  • Example(compl-dictionary)

    ctrl-x_ctrl-k

Therefore, fixed the conditional statement so that the waiting does not occur.
This patch was advised by @h-east .

Ref: vim-jp/issues#1040

@codecov-io
Copy link

codecov-io commented Mar 25, 2017

Codecov Report

Merging #1591 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1591      +/-   ##
==========================================
- Coverage   74.53%   74.53%   -0.01%     
==========================================
  Files          76       76              
  Lines      124800   124800              
==========================================
- Hits        93023    93022       -1     
- Misses      31777    31778       +1
Impacted Files Coverage Δ
src/os_unix.c 58.17% <100%> (-0.05%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a0141d...861e5be. Read the comment docs.

@brammool brammool closed this in 1572e30 Mar 25, 2017
chrisbra pushed a commit to chrisbra/vim that referenced this pull request Mar 28, 2017
Problem:    Check for available characters takes too long.
Solution:   Only check did_start_blocking if wtime is negative. (Daisuke
            Suzuki, closes vim#1591)
desvp pushed a commit to desvp/vim that referenced this pull request May 30, 2017
Problem:    Check for available characters takes too long.
Solution:   Only check did_start_blocking if wtime is negative. (Daisuke
            Suzuki, closes vim#1591)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants