Skip to content

Fix #1 for user which not compiled vim with perl#2

Merged
joonty merged 3 commits intojoonty:masterfrom
stephpy:master
Apr 23, 2012
Merged

Fix #1 for user which not compiled vim with perl#2
joonty merged 3 commits intojoonty:masterfrom
stephpy:master

Conversation

@stephpy
Copy link
Copy Markdown
Contributor

@stephpy stephpy commented Apr 19, 2012

  • Replaces tabs by spaces
  • Fix Error add_ok #1 for user which not compiled vim with perl

I replaced too

 exe ":sign place 4782 name=" . sign . " line=" . line . " file=\".expand(\"%:p\")"

by

exe ":sign place 4782 name=" . sign . " line=" . line . " file=\"".expand("%:p")."\""

I did not test it coz i did not compiled my vim with perl yet, but it seems to be good.

When i'll find time, i'll look at how to implement this for non perl vim ...

@stephpy
Copy link
Copy Markdown
Contributor Author

stephpy commented Apr 22, 2012

I see, the only one error is than line and file vars are prefixed by <ac>, i'll look at how to delete this char ...

@stephpy
Copy link
Copy Markdown
Contributor Author

stephpy commented Apr 22, 2012

I just change the position of pos var ... to not retrieve the <ac> char ... Coz i did not find how to delete this char.

Can you test it with perl extension, i would not want to break this bundle when vim is compiled with perl.

I you have any idea to substitute <ac> ...

@joonty
Copy link
Copy Markdown
Owner

joonty commented Apr 23, 2012

Thanks for this, looks good! I'll give it a go with and without Perl, and will add the latest on vim.org if everything works.

joonty added a commit that referenced this pull request Apr 23, 2012
Fix non-perl highlighting, closes #1
@joonty joonty merged commit 7a55ddb into joonty:master Apr 23, 2012
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.

Error add_ok

2 participants