Bug in @{reg} together with g@?

3 views
Skip to first unread message

Nikolai Weibull

unread,
Jan 8, 2008, 11:57:00 AM1/8/08
to vim...@googlegroups.com
I have the following mapping:

map g: <Esc>:set operatorfunc=<SID>get_command_mode_range<CR>g@

and have recorded

g:}j^M

into register 'a'.

Running @a now does nothing.

Why?

(It should run :join from the current line to the end of the current paragraph.)

Charles E Campbell Jr

unread,
Jan 8, 2008, 12:45:21 PM1/8/08
to vim...@googlegroups.com
Nikolai Weibull wrote:

It might help folks help you if you included the
get_command_mode_range() function.

Regards,
Chip Campbell

Nikolai Weibull

unread,
Jan 8, 2008, 2:57:49 PM1/8/08
to vim...@googlegroups.com
On Jan 8, 2008 6:45 PM, Charles E Campbell Jr <drc...@campbellfamily.biz> wrote:

> It might help folks help you if you included the
> get_command_mode_range() function.

Ugh, yeah, I'm beginning to have a suspicion as to what the problem is:

function! s:get_command_mode_range(type)
let b = line("'[")
let e = line("']")

if b < e
let range = '.,+' . (e - b)
elseif b == e
let range = '.'
else
let range = '.,+' . (b - e)
endif

call feedkeys(':' . range, 'n')
endfunction

I'm guessing that feedkeys() is the culprit here.

Ben Schmidt

unread,
Jan 8, 2008, 6:21:20 PM1/8/08
to vim...@googlegroups.com

You're missing a :, the <Esc> in the mapping clears the command line but doesn't
put the : back to receive the }j^M which instead become normal commands (end of
paragraph, cursor down, cursor to next line). Either record

g::}j^M

or use

map g: <Esc>:set operatorfunc=<SID>get_command_mode_range<CR>g@:

Cheers,

Ben.


Send instant messages to your online friends http://au.messenger.yahoo.com

Ben Schmidt

unread,
Jan 8, 2008, 6:25:57 PM1/8/08
to vim...@googlegroups.com
>> It might help folks help you if you included the
>> get_command_mode_range() function.
>
> Ugh, yeah, I'm beginning to have a suspicion as to what the problem is:
>
> function! s:get_command_mode_range(type)
> let b = line("'[")
> let e = line("']")
>
> if b < e
> let range = '.,+' . (e - b)
> elseif b == e
> let range = '.'
> else
> let range = '.,+' . (b - e)
> endif
>
> call feedkeys(':' . range, 'n')
> endfunction

Mmm. And excuse my dumb reply from earlier that was agnostic about your function.

Andy Wokula

unread,
Jan 8, 2008, 6:32:22 PM1/8/08
to vim...@googlegroups.com
Nikolai Weibull schrieb:

Yes, feedkeys() puts its argument at the end of the input buffer, which
already contains the contents of register a.

I think you have found a limit of g@ -- I wasn't able to find a solution
that keeps the elegance, now an additional _r in the recording is needed
to insert the range.

Try this one (g: for Normal mode only):

nmap g: :set operatorfunc=<sid>get_command_mode_range<CR>g@

let @a = "g:}_rj\r"
" note: the final ^M causes an additional ^@ to be added to the
" register

func! s:get_command_mode_range(type)


let b = line("'[")
let e = line("']")

if b < e
let range = '.,+' . (e - b)
elseif b == e
let range = '.'
else
let range = '.,+' . (b - e)
endif

exe "nmap _r :". range
endfunc


Problem:
If we use g@ in the {rhs} of a mapping for g: , we cannot add further
keys there, because Vim has no concept of place holders for the motion
the user has to type. Currently, the first few of the "further keys"
will be taken as the motion for g@ before continuing.

Fiction (can be safely ignored):
A solution (another todo item?) could be a mode in which g@ (i.e. its
new brother, gX@) evaluates the operatorfunction as if defined with
<expr>. This would change the above code into this:

nmap g: :set operatorfunc=<sid>get_command_mode_range<CR>gX@

let @a = "g:}j\r"
" ah, without _r

func! s:get_command_mode_range(type)


let b = line("'[")
let e = line("']")

if b < e
let range = '.,+' . (e - b)
elseif b == e
let range = '.'
else
let range = '.,+' . (b - e)
endif

return ":". range
endfunc

--
Andy

Andy Wokula

unread,
Jan 14, 2008, 7:28:27 AM1/14/08
to vim...@googlegroups.com
Andy Wokula schrieb:

Ok, I was wrong. Vim *has* inputsave() and inputrestore().

nmap <silent> g: :set operatorfunc=<sid>ExRangeOp<CR>g@

let @a = "g:}join"
" final <CR> to be added

func! s:ExRangeOp(type)


let b = line("'[")
let e = line("']")

if b < e
let range = '.,+' . (e - b)
elseif b == e
let range = '.'
else
let range = '.,+' . (b - e)
endif

call inputsave()
call feedkeys(':'. range. "\<c-r>=''[inputrestore()]\r", 'n')
endfunc

--
Andy

Reply all
Reply to author
Forward
0 new messages