Commit: runtime(dosini): save and restore cpo value in syntax script

10 views
Skip to first unread message

Christian Brabandt

unread,
Aug 21, 2023, 1:00:10 AM8/21/23
to vim...@googlegroups.com
runtime(dosini): save and restore cpo value in syntax script

Commit: https://github.com/vim/vim/commit/690963924956d800b94bb86076aa9d25f04565ac
Author: Christian Brabandt <c...@256bit.org>
Date: Mon Aug 21 06:49:38 2023 +0200

runtime(dosini): save and restore cpo value in syntax script

Commit dd0ad2598898c2b4641c4acd5b70b6184fa698ed introduced
line-continuation. However, to make sure this does not cause an error
when Vim is run in compatible mode, we need to set compatibility mode
temporarily and reset it back when finished reading the file.

This fixes: https://groups.google.com/g/vim_use/c/9zccgo_RIqM/m/xlUmhBktBgAJ

Signed-off-by: Christian Brabandt <c...@256bit.org>

diff --git a/runtime/syntax/dosini.vim b/runtime/syntax/dosini.vim
index 751a12f4b..66e17ec9a 100644
--- a/runtime/syntax/dosini.vim
+++ b/runtime/syntax/dosini.vim
@@ -6,7 +6,7 @@
" Current Maintainer: Hong Xu <ho...@topbug.net>
" Homepage: http://www.vim.org/scripts/script.php?script_id=3747
" Repository: https://github.com/xuhdev/syntax-dosini.vim
-" Last Change: 2023 Jun 27
+" Last Change: 2023 Aug 20


" quit when a syntax file was already loaded
@@ -14,6 +14,10 @@ if exists("b:current_syntax")
finish
endif

+" using of line-continuation requires cpo&vim
+let s:cpo_save = &cpo
+set cpo&vim
+
" shut case off
syn case ignore

@@ -39,4 +43,7 @@ hi def link dosiniValue String

let b:current_syntax = "dosini"

+let &cpo = s:cpo_save
+unlet s:cpo_save
+
" vim: sts=2 sw=2 et

Yegappan Lakshmanan

unread,
Aug 21, 2023, 1:24:42 AM8/21/23
to vim...@googlegroups.com
Hi,

On Sun, Aug 20, 2023 at 10:00 PM Christian Brabandt <cbl...@256bit.org> wrote:
>
> runtime(dosini): save and restore cpo value in syntax script
>

A similar change is needed in the runtime/syntax/context.vim file also.

I used the following command to find this:

$ cd vim/runtime/syntax
$ egrep '^ +\\' `grep -L cpo *.vim`

Regards,
Yegappan

Christian Brabandt

unread,
Aug 21, 2023, 1:51:24 AM8/21/23
to vim...@googlegroups.com

On So, 20 Aug 2023, Yegappan Lakshmanan wrote:

> On Sun, Aug 20, 2023 at 10:00 PM Christian Brabandt <cbl...@256bit.org> wrote:
> > runtime(dosini): save and restore cpo value in syntax script
>
> A similar change is needed in the runtime/syntax/context.vim file also.
>
> I used the following command to find this:
>
> $ cd vim/runtime/syntax
> $ egrep '^ +\\' `grep -L cpo *.vim`

Oh but that is using vim9script. Does that make a difference?

Best,
Christian
--
There is a multi-legged creature crawling on your shoulder.
-- Spock, "A Taste of Armageddon", stardate 3193.9

Yegappan Lakshmanan

unread,
Aug 21, 2023, 9:56:26 AM8/21/23
to vim...@googlegroups.com
Hi Christian,

On Sun, Aug 20, 2023 at 10:51 PM Christian Brabandt <cbl...@256bit.org> wrote:
>
>
> On So, 20 Aug 2023, Yegappan Lakshmanan wrote:
>
> > On Sun, Aug 20, 2023 at 10:00 PM Christian Brabandt <cbl...@256bit.org> wrote:
> > > runtime(dosini): save and restore cpo value in syntax script
> >
> > A similar change is needed in the runtime/syntax/context.vim file also.
> >
> > I used the following command to find this:
> >
> > $ cd vim/runtime/syntax
> > $ egrep '^ +\\' `grep -L cpo *.vim`
>
> Oh but that is using vim9script. Does that make a difference?
>

Yes. I missed that it is written in vim9script. In that case, 'cpo'
need not be set and modified.

Regards,
Yegappan
Reply all
Reply to author
Forward
0 new messages