Skip to content

Commit

Permalink
Update fsi_extra_*_parameters handling
Browse files Browse the repository at this point in the history
Removing the default for `fsi_extra_parameters` causes an error when
concatenating command line args for the interpreter. Wrap in a
conditional check for existence. Add a warning message to migrate to the
new parameters

Implements #82
  • Loading branch information
greggyb committed May 23, 2024
1 parent 6c65259 commit fb146db
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions autoload/fsharp.vim
Original file line number Diff line number Diff line change
Expand Up @@ -556,9 +556,12 @@ endfunction

function! s:get_fsi_command()
let cmd = g:fsharp#fsi_command
for prm in g:fsharp#fsi_extra_parameters
let cmd = cmd . " " . prm
endfor
if exists("g:fsharp#fsi_extra_parameters")
echom "[Ionide-vim]: `g:fsharp#fsi_extra_parameters` is being deprecated. Migrate to `g:fsharp#fsi_extra_interactive_parameters` and `g:fsharp_extra_shared_parameters`."
for prm in g:fsharp#fsi_extra_parameters
let cmd = cmd . " " . prm
endfor
endif
for prm in g:fsharp#fsi_extra_interactive_parameters
let cmd = cmd . " " . prm
endfor
Expand Down

0 comments on commit fb146db

Please sign in to comment.