From: Bram Moolenaar Date: Sat, 22 Aug 2020 13:19:18 +0000 (+0200) Subject: patch 8.2.1507: using malloc() directly X-Git-Tag: v8.2.1507 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=51b6eb47b3c41b01a5559b099e65354c8897093e;p=vim patch 8.2.1507: using malloc() directly Problem: Using malloc() directly. Solution: Use ALLOC_ONE(). Remove superfluous typecast. (Hussam al-Homsi, closes #6768) --- diff --git a/src/eval.c b/src/eval.c index cc044a4bd..4b1e0f19b 100644 --- a/src/eval.c +++ b/src/eval.c @@ -4352,7 +4352,8 @@ set_ref_in_item( } else { - ht_stack_T *newitem = (ht_stack_T*)malloc(sizeof(ht_stack_T)); + ht_stack_T *newitem = ALLOC_ONE(ht_stack_T); + if (newitem == NULL) abort = TRUE; else @@ -4378,8 +4379,8 @@ set_ref_in_item( } else { - list_stack_T *newitem = (list_stack_T*)malloc( - sizeof(list_stack_T)); + list_stack_T *newitem = ALLOC_ONE(list_stack_T); + if (newitem == NULL) abort = TRUE; else diff --git a/src/memline.c b/src/memline.c index 084a8b0fa..1c594d453 100644 --- a/src/memline.c +++ b/src/memline.c @@ -5448,8 +5448,7 @@ ml_updatechunk( chunksize_T *t_chunksize = buf->b_ml.ml_chunksize; buf->b_ml.ml_numchunks = buf->b_ml.ml_numchunks * 3 / 2; - buf->b_ml.ml_chunksize = (chunksize_T *) - vim_realloc(buf->b_ml.ml_chunksize, + buf->b_ml.ml_chunksize = vim_realloc(buf->b_ml.ml_chunksize, sizeof(chunksize_T) * buf->b_ml.ml_numchunks); if (buf->b_ml.ml_chunksize == NULL) { diff --git a/src/version.c b/src/version.c index 575f343ea..bb1624f6e 100644 --- a/src/version.c +++ b/src/version.c @@ -754,6 +754,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 1507, /**/ 1506, /**/ diff --git a/src/vimrun.c b/src/vimrun.c index 26c4aa4c6..9a4c076d6 100644 --- a/src/vimrun.c +++ b/src/vimrun.c @@ -77,7 +77,7 @@ main(void) if (cmdlen >= 2 && p[0] == L'"' && p[cmdlen - 1] == L'"') { cmdlen += 3; - cmd = (wchar_t *)malloc(cmdlen * sizeof(wchar_t)); + cmd = malloc(cmdlen * sizeof(wchar_t)); if (cmd == NULL) { perror("vimrun malloc(): ");