aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-10-24 08:54:47 -04:00
Code Activity

patch 8.2.2959: sound_playfile() is not tested on MS-Windows

Browse Source 
Problem:    sound_playfile() is not tested on MS-Windows.
Solution:   Make it work and enable the test. (Dominique Pellé, closes #8338)
This commit is contained in:
Dominique Pelle
2021-06-07 20:28:45 +02:00
committed by Bram Moolenaar
parent f7a023e580
commit 2f9c209ba0
3 changed files with 13 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/sound.c
View File

@@ -355,10 +355,9 @@ f_sound_playevent(typval_T *argvars, typval_T *rettv)
if (wp == NULL)
return;
PlaySoundW(wp, NULL, SND_ASYNC | SND_ALIAS);
free(wp);
if (PlaySoundW(wp, NULL, SND_ASYNC | SND_ALIAS))
rettv->vval.v_number = ++sound_id;
free(wp);
}
void

8
src/testdir/test_sound.vim
View File

@@ -75,17 +75,15 @@ func Test_play_silent()
endfunc
func Test_play_event_error()
" Do not run test on Windows as:
" - playing event with callback is not supported on Windows.
" - FIXME: even without callback, sound_playevent('') does not return 0 on Windows. Bug?
CheckNotMSWindows
" FIXME: sound_playevent() doesn't return 0 in case of error on Windows.
if !has('win32')
call assert_equal(0, sound_playevent(''))
call assert_equal(0, sound_playevent(test_null_string()))
call assert_equal(0, sound_playevent('doesnotexist'))
call assert_equal(0, sound_playevent('doesnotexist', 'doesnotexist'))
call assert_equal(0, sound_playevent(test_null_string(), test_null_string()))
call assert_equal(0, sound_playevent(test_null_string(), test_null_function()))
endif
call assert_equal(0, sound_playfile(''))
call assert_equal(0, sound_playfile(test_null_string()))

2
src/version.c
View File

@@ -750,6 +750,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
2959,
/**/
2958,
/**/