Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Throw error if setcellwidths() invalidates 'listchars' or 'fillchars' #9024

Closed
wants to merge 1 commit into from

Conversation

zeertzjq
Copy link
Member

No description provided.

@zeertzjq
Copy link
Member Author

I think some documentation should be added about this, but I cannot decide how.

@zeertzjq zeertzjq force-pushed the setcellwidths-conflict branch 3 times, most recently from 85292d1 to eb4b3d7 Compare October 20, 2021 01:55
@codecov
Copy link

codecov bot commented Oct 20, 2021

Codecov Report

Merging #9024 (d1aeda6) into master (8b0e62c) will decrease coverage by 87.60%.
The diff coverage is 0.00%.

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

@@             Coverage Diff             @@
##           master    #9024       +/-   ##
===========================================
- Coverage   90.06%    2.45%   -87.61%     
===========================================
  Files         151      149        -2     
  Lines      168477   166109     -2368     
===========================================
- Hits       151736     4085   -147651     
- Misses      16741   162024   +145283     
Flag Coverage Δ
huge-clang-none ?
huge-gcc-none ?
huge-gcc-testgui ?
huge-gcc-unittests 2.45% <0.00%> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
src/mbyte.c 20.17% <0.00%> (-60.40%) ⬇️
src/float.c 0.00% <0.00%> (-99.22%) ⬇️
src/gui_gtk_f.c 0.00% <0.00%> (-97.43%) ⬇️
src/crypt_zip.c 0.00% <0.00%> (-97.06%) ⬇️
src/cmdhist.c 0.00% <0.00%> (-97.00%) ⬇️
src/sha256.c 0.00% <0.00%> (-96.94%) ⬇️
src/evalbuffer.c 0.00% <0.00%> (-96.88%) ⬇️
src/textprop.c 0.00% <0.00%> (-96.84%) ⬇️
src/match.c 0.00% <0.00%> (-96.76%) ⬇️
src/libvterm/src/rect.h 0.00% <0.00%> (-96.56%) ⬇️
... and 138 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 8b0e62c...c92422b. Read the comment docs.

@zeertzjq zeertzjq closed this Oct 20, 2021
@zeertzjq zeertzjq reopened this Oct 20, 2021
@zeertzjq
Copy link
Member Author

I think some documentation should be added about this, but I cannot decide how.

Maybe move E834 and E835 tags to somewhere else?

@zeertzjq zeertzjq changed the title Prevent setcellwidths() from invalidating 'listchars' and 'fillchars' Throw error if setcellwidths() invalidates 'listchars' or 'fillchars' Oct 20, 2021
@brammool brammool closed this in 94358a1 Oct 20, 2021
@zeertzjq zeertzjq deleted the setcellwidths-conflict branch October 20, 2021 10:10
delphinus added a commit to delphinus/neovim that referenced this pull request Aug 6, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1
delphinus added a commit to delphinus/neovim that referenced this pull request Aug 6, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1
delphinus added a commit to delphinus/neovim that referenced this pull request Aug 6, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1
delphinus added a commit to delphinus/neovim that referenced this pull request Aug 6, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1
delphinus added a commit to delphinus/neovim that referenced this pull request Aug 7, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Aug 8, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1

Cherry-pick f_setcellwidths() change from patch 9.0.0036.
Cherry-pick 'ambiwidth' docs update from runtime update 079ba76ae7a7.
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Aug 8, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1

Cherry-pick f_setcellwidths() change from patch 9.0.0036.
Cherry-pick 'ambiwidth' docs update from runtime update 079ba76ae7a7.
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Aug 8, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1

Cherry-pick f_setcellwidths() change from patch 9.0.0036.
Cherry-pick 'ambiwidth' docs update from runtime update 079ba76ae7a7.
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Aug 8, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1

Cherry-pick f_setcellwidths() change from patch 9.0.0036.
Cherry-pick 'ambiwidth' docs update from runtime update 079ba76ae7a7.
smjonas pushed a commit to smjonas/neovim that referenced this pull request Dec 31, 2022
…s' invalid

Problem:    setcellwidths() may make 'listchars' or 'fillchars' invalid.
Solution:   Check the value and give an error. (closes vim/vim#9024)
vim/vim@94358a1

Cherry-pick f_setcellwidths() change from patch 9.0.0036.
Cherry-pick 'ambiwidth' docs update from runtime update 079ba76ae7a7.
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.

1 participant