Skip to content

Conversation

pmoura
Copy link
Contributor

@pmoura pmoura commented Dec 15, 2023

This pull request also fixes some syntax highlight bugs.

command -nargs=+ HiLink hi link <args>
else
command -nargs=+ HiLink hi def link <args>
endif
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This HiLink compatibility command was removed from the distributed runtime files about eight years ago. So, the original file using hi def link is actually the correct approach.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick feedback. Fixed.


if version < 600
syntax clear
elseif exists("b:current_syntax")
finish
endif

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Likewise, the original file is the correct approach here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@pmoura pmoura force-pushed the update_logtalk_support branch from 86234dc to c2a56d5 Compare December 16, 2023 09:54
@chrisbra
Copy link
Member

thanks!

@chrisbra chrisbra merged commit 0f61943 into vim:master Dec 16, 2023
clason added a commit to clason/neovim that referenced this pull request Dec 16, 2023
runtime(logtalk): Update Logtalk runtime files for the latest language spec (vim/vim#13697)

vim/vim@0f61943

Co-authored-by: Paulo Moura <pmoura@logtalk.org>
clason added a commit to neovim/neovim that referenced this pull request Dec 16, 2023
runtime(logtalk): Update Logtalk runtime files for the latest language spec (vim/vim#13697)

vim/vim@0f61943

Co-authored-by: Paulo Moura <pmoura@logtalk.org>
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