navid.sassan/vim-tmux-runner.vim
1
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't base pane validity on count

Browse Source 
This introduced an implicit dependency on zero-based pane indexing.
This code now checks that the desired pane is one of the current pane
indexes.

Closes #42 - Off by one error in runner_pane with base-index 1 in tmux
This commit is contained in:
Chris Toomey 2015-01-05 20:05:30 -05:00
parent d56939fa94
commit 8d064375e5
1 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
plugin/vim-tmux-runner.vim
View File

@ -95,7 +95,7 @@ function! s:KillRunnerPane()
endfunction
function! s:ActivePaneIndex()
return s:SendTmuxCommand("display-message -p \"#{pane_index}\"")
return str2nr(s:SendTmuxCommand("display-message -p \"#{pane_index}\""))
endfunction
function! s:TmuxPanes()
@ -229,7 +229,7 @@ function! s:PromptForRunnerToAttach()
endif
echohl String | let desired_pane = input('Pane #: ') | echohl None
if desired_pane != ''
call s:AttachToPane(str2nr(desired_pane))
call s:AttachToPane(desired_pane)
else
call s:EchoError("No pane specified. Cancelling.")
endif
@ -244,13 +244,14 @@ function! s:CurrentMajorOrientation()
endfunction
function! s:AttachToPane(desired_pane)
if s:ValidRunnerPaneNumber(a:desired_pane)
let s:runner_pane = a:desired_pane
let desired_pane = str2nr(a:desired_pane)
if s:ValidRunnerPaneNumber(desired_pane)
let s:runner_pane = desired_pane
let s:vim_pane = s:ActivePaneIndex()
let s:vtr_orientation = s:CurrentMajorOrientation()
echohl String | echo "\rRunner pane set to: " . a:desired_pane | echohl None
echohl String | echo "\rRunner pane set to: " . desired_pane | echohl None
else
call s:EchoError("Invalid pane number: " . a:desired_pane)
call s:EchoError("Invalid pane number: " . desired_pane)
endif
endfunction
@ -258,9 +259,13 @@ function! s:EchoError(message)
echohl ErrorMsg | echo "\rVTR: ". a:message | echohl None
endfunction
function! s:DesiredPaneExists(desired_pane)
return count(s:PaneIndices(), a:desired_pane) == 0
endfunction
function! s:ValidRunnerPaneNumber(desired_pane)
if a:desired_pane == s:ActivePaneIndex() | return 0 | endif
if a:desired_pane > (s:PaneCount() - 1) | return 0 | endif
if s:DesiredPaneExists(a:desired_pane) | return 0 | endif
return 1
endfunction