[m-rev.] for review: rewrite of vim syntax file

Peter Wang novalazy at gmail.com
Wed Apr 15 12:00:06 AEST 2015


On Tue, 14 Apr 2015 15:26:09 +0800, Sebastian Godelet <sebastian.godelet at outlook.com> wrote:
>
> Would you mind rebasing your diff to the current master (that one has 
> the colouring for head clauses already addressed) if possible?
> If that's not possible, then I'll do it by hand.

It's here: https://github.com/wangp/mercury-experimental/tree/vim

> 
> In your diff, you marked these <Fx> keyboard commands as conflicting:
>     " <F6> attempts to wrap a call up with { } braces for DCG escapes.
>     "
> -nnoremap <F6> I{ <ESC>%a }<ESC>j
> +" pw - conflicts
> +" nnoremap <F6> I{ <ESC>%a }<ESC>j

Sorry, it only conflicts with my own <F6> binding.

>     " <F7> and <F8> comment and uncomment lines out respectively.
>     "
> -nnoremap <F7> 0i% <ESC>j
> -nnoremap <F8> :s/% //e<CR>j
> +" pw - should only be applicable to mercury
> +" nnoremap <F7> 0i% <ESC>j
> +" nnoremap <F8> :s/% //e<CR>j
> 
> These I didn't touch (at least I can't recall that), as far as I 
> remember there were like this in the original version.

Never mind, I just use something else for toggling comments which works
for multiple languages.

> Also about the comment line length, I just adjusted the line length to 
> the one in the standard library, e.g. from io.m (77):
> %---------------------------------------------------------------------------%
> Before it was 80 chars but I remember that I once read on the mailing 
> list that this was deemed to long once.
> Maybe I just miscount. Hope you can clarify on this.

I see, the old plugin had it at 80 chars but I had modified my copy ages
ago to 79 chars.  Now some files are moving to 77 chars.  Fine.

Peter



More information about the reviews mailing list