Skip to content

Commit

Permalink
patch 8.1.0978: blob not tested with Perl
Browse files Browse the repository at this point in the history
Problem:    Blob not tested with Perl.
Solution:   Add more test coverage.  Fixes a crash. (Dominique Pelle,
            closes #4037)
  • Loading branch information
brammool committed Feb 23, 2019
1 parent 0d13cce commit 2472ae8
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/if_perl.xs
Expand Up @@ -1570,7 +1570,7 @@ Blob(SV* sv)
newsv = newSVpv("0z", 2);
for (i = 0; i < len; i++)
{
sprintf(buf, "%02X", s[i]);
sprintf(buf, "%02X", (unsigned char)(s[i]));
sv_catpvn(newsv, buf, 2);
}
RETVAL = newsv;
Expand Down
7 changes: 7 additions & 0 deletions src/testdir/test_perl.vim
Expand Up @@ -29,6 +29,13 @@ EOF
call assert_equal('abc/def/', getline('$'))
endfunc

funct Test_VIM_Blob()
call assert_equal('0z', perleval('VIM::Blob("")'))
call assert_equal('0z31326162', perleval('VIM::Blob("12ab")'))
call assert_equal('0z00010203', perleval('VIM::Blob("\x00\x01\x02\x03")'))
call assert_equal('0z8081FEFF', perleval('VIM::Blob("\x80\x81\xfe\xff")'))
endfunc

func Test_buffer_Delete()
new
call setline(1, ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h'])
Expand Down
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -779,6 +779,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
978,
/**/
977,
/**/
Expand Down

0 comments on commit 2472ae8

Please sign in to comment.