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

E239: Invalid sign text on vim 8.1.320 #2452

Closed
mgedmin opened this issue Apr 23, 2019 · 3 comments
Closed

E239: Invalid sign text on vim 8.1.320 #2452

mgedmin opened this issue Apr 23, 2019 · 3 comments
Labels

Comments

@mgedmin
Copy link
Contributor

mgedmin commented Apr 23, 2019

Information

VIM version

VIM - Vi IMproved 8.1 (2018 May 18, compiled Nov 03 2018 00:15:14)
Included patches: 1-320

Operating System: Ubuntu 19.04

What went wrong

I run /usr/bin/vim and I keep getting briefly flashing error messages that say

Error detected while processing /home/mg/.vim/bundle/ale/autoload/ale/sign.vim:
line   74:
E239: Invalid sign text: ☞\ 
Error detected while processing function <SNR>126_VimCloseCallback:
line   11:
E171: Missing :endif

This doesn't happen when I use vim 8.1.1198 that I built from source.

This is probably related to #2351, but appears to be Vim version specific.

I've been using signs with trailing spaces since April 2018, and they worked fine with the versions of Vim and ALE available back then.

I keep upgrading both Vim and ALE periodically. At some point I started seeing these errors, then they were gone and now I get them when I accidentally launch the system vim instead of my own vim.

Probably related vim commits:

Probably related ale commits:

Suggestion: do the backslash escaping only if vim's version is 8.1.0588 or greater

Reproducing the bug

  1. Add let g:ale_sign_error = '☞ ' to your .vimrc
  2. Open a file with lint errors using an older Vim version
@mgedmin
Copy link
Contributor Author

mgedmin commented Apr 23, 2019

Workaround: dropping the trailing space from g:ale_sign_error makes things work with both old and new Vim versions.

(I only added the space because without it the double-width hand overlapped with the closing ' when I was editing my .vimrc because terminals are terrible.)

mgedmin added a commit to mgedmin/dotvim that referenced this issue Apr 23, 2019
This error happens when I use new ALE with old Vim versions.
See dense-analysis/ale#2452
@w0rp w0rp added the bug label Apr 23, 2019
@w0rp w0rp closed this as completed in 893ac34 Apr 23, 2019
@w0rp
Copy link
Member

w0rp commented Apr 23, 2019

I have fixed this now by automatically removing trailing spaces from settings for sign text. I agree that what worked before here should continue to work.

@mgedmin
Copy link
Contributor Author

mgedmin commented Apr 23, 2019

You're awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants