Skip to content

Conversation

puremourning
Copy link
Contributor

Fixes #4153

Tests fail prior to change and pass afterwards.

I found that the check before calling py_fix_cursor is not consistently applied, so this patch applies it more consistently.

@puremourning puremourning changed the title Fix E315 when writing to hidden buffer using python buffer object WIP: Fix E315 when writing to hidden buffer using python buffer object Mar 23, 2019
@puremourning
Copy link
Contributor Author

WIP because tests failing on py2. Working on it.

@puremourning puremourning changed the title WIP: Fix E315 when writing to hidden buffer using python buffer object Fix E315 when writing to hidden buffer using python buffer object Mar 23, 2019
@puremourning
Copy link
Contributor Author

Fixed the tests (hopefully)

@codecov-io
Copy link

Codecov Report

Merging #4154 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4154      +/-   ##
==========================================
+ Coverage   79.23%   79.25%   +0.02%     
==========================================
  Files         105      105              
  Lines      141092   141094       +2     
==========================================
+ Hits       111792   111826      +34     
+ Misses      29300    29268      -32
Impacted Files Coverage Δ
src/if_py_both.h 76.59% <100%> (+1.18%) ⬆️
src/if_xcmdsrv.c 83.48% <0%> (-0.72%) ⬇️
src/window.c 83.54% <0%> (-0.04%) ⬇️
src/gui.c 58.1% <0%> (+0.1%) ⬆️

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 c3e92c1...a24e696. Read the comment docs.

@brammool brammool closed this in 63dbfd3 Mar 23, 2019
janlazo added a commit to janlazo/neovim that referenced this pull request Mar 23, 2019
…ffer

Problem:    E315 ml_get error when using Python and hidden buffer.
Solution:   Make sure the cursor position is valid. (Ben Jackson,
            closes vim/vim#4153, closes vim/vim#4154)
vim/vim@63dbfd3
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.

E315: ml_get: invalid lnum when writing to buffer in no windows using python API
2 participants