Skip to content

Add CursorLineSign and CursorLineFold highlight groups#9201

Closed
gpanders wants to merge 1 commit intovim:masterfrom
gpanders:cursorlinesign
Closed

Add CursorLineSign and CursorLineFold highlight groups#9201
gpanders wants to merge 1 commit intovim:masterfrom
gpanders:cursorlinesign

Conversation

@gpanders
Copy link
Contributor

These are similar to the SignColumn and FoldColumn groups but only take
effect for the cursor line and when the 'cursorline' option is set.

@codecov
Copy link

codecov bot commented Nov 23, 2021

Codecov Report

Merging #9201 (819c16e) into master (cf1e023) will decrease coverage by 1.73%.
The diff coverage is 88.63%.

❗ Current head 819c16e differs from pull request most recent head ffa66bb. Consider uploading reports for the commit ffa66bb to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9201      +/-   ##
==========================================
- Coverage   90.17%   88.44%   -1.74%     
==========================================
  Files         151      150       -1     
  Lines      170849   167991    -2858     
==========================================
- Hits       154070   148576    -5494     
- Misses      16779    19415    +2636     
Flag Coverage Δ
huge-clang-none ?
huge-gcc-none ?
huge-gcc-testgui 88.40% <88.63%> (-0.01%) ⬇️
huge-gcc-unittests 2.46% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/highlight.c 86.30% <ø> (-4.05%) ⬇️
src/sign.c 94.57% <85.18%> (-0.44%) ⬇️
src/drawline.c 85.80% <93.75%> (-0.08%) ⬇️
src/popupwin.c 94.47% <100.00%> (-0.20%) ⬇️
src/libvterm/src/rect.h 0.00% <0.00%> (-96.56%) ⬇️
src/libvterm/src/mouse.c 0.00% <0.00%> (-48.34%) ⬇️
src/libvterm/src/state.c 48.87% <0.00%> (-40.75%) ⬇️
src/libvterm/include/vterm.h 0.00% <0.00%> (-37.50%) ⬇️
src/libvterm/src/keyboard.c 54.73% <0.00%> (-33.69%) ⬇️
src/libvterm/src/pen.c 51.03% <0.00%> (-33.63%) ⬇️
... and 111 more

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 cf1e023...ffa66bb. Read the comment docs.

@brammool
Copy link
Contributor

Asan reports a couple of memory leaks, please fix that.

@gpanders
Copy link
Contributor Author

Forgot a vim_free in sign_define_from_dict. Should be fixed now.

@gpanders
Copy link
Contributor Author

These tests are failing:

From test_alot.vim:
	Found errors in Test_very_large_count_64bit():
	command line..script /home/runner/work/vim/vim/src/testdir/runtest.vim[455]..function RunTheTest[44]..Test_very_large_count_64bit line 7: command did not fail: norm 44444444444444p
	Found errors in Test_very_large_count_block_64bit():
	command line..script /home/runner/work/vim/vim/src/testdir/runtest.vim[455]..function RunTheTest[44]..Test_very_large_count_block_64bit line 8: command did not fail: norm 44444444444444p

Not sure how or if that's related to these changes though.

@chrisbra
Copy link
Member

looks like those currently also happen on master, so probably unrelated

@brammool
Copy link
Contributor

Please sync to head, then the failures should be gone.

These are similar to the SignColumn and FoldColumn groups but only take
effect for the cursor line and when the 'cursorline' option is set.
@gpanders
Copy link
Contributor Author

Done

@brammool brammool closed this in e413ea0 Nov 24, 2021
@gpanders gpanders deleted the cursorlinesign branch November 24, 2021 17:10
jamessan added a commit to jamessan/neovim that referenced this pull request Dec 6, 2021
Problem:    Cannot adjust sign highlighting for 'cursorline'.
Solution:   Add CursorLineSign and CursorLineFold highlight groups.
            (Gregory Anders, closes vim/vim#9201)
vim/vim@e413ea0
jamessan added a commit to jamessan/neovim that referenced this pull request Dec 6, 2021
Problem:    Cannot adjust sign highlighting for 'cursorline'.
Solution:   Add CursorLineSign and CursorLineFold highlight groups.
            (Gregory Anders, closes vim/vim#9201)
vim/vim@e413ea0
jamessan added a commit to jamessan/neovim that referenced this pull request Dec 6, 2021
Problem:    Cannot adjust sign highlighting for 'cursorline'.
Solution:   Add CursorLineSign and CursorLineFold highlight groups.
            (Gregory Anders, closes vim/vim#9201)
vim/vim@e413ea0
jamessan added a commit to jamessan/neovim that referenced this pull request Dec 9, 2021
Problem:    Cannot adjust sign highlighting for 'cursorline'.
Solution:   Add CursorLineSign and CursorLineFold highlight groups.
            (Gregory Anders, closes vim/vim#9201)
vim/vim@e413ea0
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.

3 participants