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

version.c: mark NA patches #5380

Merged
merged 1 commit into from Oct 6, 2016
Merged

version.c: mark NA patches #5380

merged 1 commit into from Oct 6, 2016

Conversation

Shougo
Copy link
Contributor

@Shougo Shougo commented Sep 23, 2016

NA list:

1653: matchit packadd
1750: channel
1770: termguicolors
1794: GUI Win32
1805: Makefile
1804, 1945: manpager.vim
1811: channel
2115, 2232, 2278, 2319: defaults.vim

included:

1763: spell

@Shougo
Copy link
Contributor Author

Shougo commented Oct 2, 2016

I have updated the list.

@jamessan
Copy link
Member

jamessan commented Oct 3, 2016

1763: spell

Just because it's a spelling change doesn't mean it should be excluded. We still use the spell checking code.

This one in particular silences a Coverity issue, since the corresponding code is already excluded by an #if 0.

@Shougo
Copy link
Contributor Author

Shougo commented Oct 3, 2016

I have found the #if 0 is already added.

@jamessan
Copy link
Member

jamessan commented Oct 4, 2016

I have found the #if 0 is already added.

Hmm, so it is. I must've been looking at the wrong code.

In general, I'd probably uncomment the number for a patch where we already made the change, instead of marking it NA. It's not significant in this case, but it would be if it were something that someone would want to use has('patch...') with.

NA list:

1653: matchit packadd
1750: channel
1770: termguicolors
1794: GUI Win32
1805: Makefile
1804, 1945: manpager.vim
1811: channel
2115, 2232, 2278, 2319: defaults.vim

included:
1763: spell
@Shougo
Copy link
Contributor Author

Shougo commented Oct 4, 2016

I have updated.
7.4.1763 is already included.

@justinmk justinmk merged commit 2ef1b68 into neovim:master Oct 6, 2016
@Shougo Shougo deleted the version branch October 6, 2016 14:27
@Shougo
Copy link
Contributor Author

Shougo commented Oct 6, 2016

Thank you!

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.

None yet

3 participants