mirror of
https://github.com/neovim/neovim.git
synced 2025-02-25 18:55:25 -06:00
clipboard: handle/avoid SIGTERM with previous owner #10765
Fixes regression due to signal being reported with exit status.
ref #10573 939d9053bd
ref https://github.com/neovim/neovim/issues/7054#issuecomment-520282429
This commit is contained in:
parent
8fda095b6d
commit
2fafed6bb8
@ -159,9 +159,7 @@ function! s:clipboard.set(lines, regtype, reg) abort
|
||||
end
|
||||
|
||||
if s:selections[a:reg].owner > 0
|
||||
" The previous provider instance should exit when the new one takes
|
||||
" ownership, but kill it to be sure we don't fill up the job table.
|
||||
call jobstop(s:selections[a:reg].owner)
|
||||
let prev_job = s:selections[a:reg].owner
|
||||
end
|
||||
let s:selections[a:reg] = copy(s:selection)
|
||||
let selection = s:selections[a:reg]
|
||||
@ -175,13 +173,23 @@ function! s:clipboard.set(lines, regtype, reg) abort
|
||||
call jobsend(jobid, a:lines)
|
||||
call jobclose(jobid, 'stdin')
|
||||
let selection.owner = jobid
|
||||
let ret = 1
|
||||
else
|
||||
echohl WarningMsg
|
||||
echomsg 'clipboard: failed to execute: '.(s:copy[a:reg])
|
||||
echohl None
|
||||
return 0
|
||||
let ret = 1
|
||||
endif
|
||||
return 1
|
||||
|
||||
" The previous provider instance should exit when the new one takes
|
||||
" ownership, but kill it to be sure we don't fill up the job table.
|
||||
if exists('prev_job')
|
||||
call timer_start(1000, {... ->
|
||||
\ jobwait([prev_job], 0)[0] == -1
|
||||
\ && jobstop(prev_job)})
|
||||
endif
|
||||
|
||||
return ret
|
||||
endfunction
|
||||
|
||||
function! provider#clipboard#Call(method, args) abort
|
||||
|
Loading…
Reference in New Issue
Block a user