Problem: Unnecessary type casts.
Solution: Remove type casts from alloc() and lalloc() calls. (Mike Williams)
int id)
{
cbq_T *head = &channel->ch_part[part].ch_cb_head;
- cbq_T *item = (cbq_T *)alloc((int)sizeof(cbq_T));
+ cbq_T *item = (cbq_T *)alloc(sizeof(cbq_T));
if (item != NULL)
{
}
else
{
- writeq_T *last = (writeq_T *)alloc((int)sizeof(writeq_T));
+ writeq_T *last = (writeq_T *)alloc(sizeof(writeq_T));
if (last != NULL)
{
char_u *seed,
int seed_len)
{
- cryptstate_T *state = (cryptstate_T *)alloc((int)sizeof(cryptstate_T));
+ cryptstate_T *state = (cryptstate_T *)alloc(sizeof(cryptstate_T));
if (state == NULL)
return state;
/* Not buffering, just return EOF. */
return (long)len;
- *newptr = alloc((long)len);
+ *newptr = alloc(len);
if (*newptr == NULL)
return -1;
method->encode_fn(state, from, len, *newptr);
dict_alloc_id(alloc_id_T id UNUSED)
{
#ifdef FEAT_EVAL
- if (alloc_fail_id == id && alloc_does_fail((long_u)sizeof(list_T)))
+ if (alloc_fail_id == id && alloc_does_fail(sizeof(list_T)))
return NULL;
#endif
return (dict_alloc());
|| strchr(cp, '/') != NULL)
return;
- tmpname = alloc((int)strlen(cp) + 1);
+ tmpname = alloc(strlen(cp) + 1);
strcpy(tmpname, cp);
tmpname[strlen(tmpname) - 1] = 'x'; /* .exe -> .exx */
if (choices[idx].text != NULL)
free(choices[idx].text);
- choices[idx].text = alloc((int)(strlen(fmt) + strlen(arg)) - 1);
+ choices[idx].text = alloc(strlen(fmt) + strlen(arg) - 1);
sprintf(choices[idx].text, fmt, arg);
}
s = p + strlen(p);
if (names != NULL)
{
- names[count] = alloc((int)(s - p) + 1);
+ names[count] = alloc(s - p + 1);
strncpy(names[count], p, s - p);
names[count][s - p] = NUL;
}
}
if (names != NULL)
break;
- names = alloc((int)(count + 1) * sizeof(char *));
+ names = alloc((count + 1) * sizeof(char *));
}
names[0] = alloc(50);
sprintf(names[0], "Select directory to create %s in:", name);
* would also work, but some plugins depend on the name being
* printable text. */
sprintf(sid_buf, "<SNR>%ld_", (long)current_sctx.sc_sid);
- name = alloc((int)(STRLEN(sid_buf) + STRLEN(s + off) + 1));
+ name = alloc(STRLEN(sid_buf) + STRLEN(s + off) + 1);
if (name != NULL)
{
STRCPY(name, sid_buf);
}
/* Make an array with each entry pointing to an item in the List. */
- ptrs = (sortItem_T *)alloc((int)(len * sizeof(sortItem_T)));
+ ptrs = (sortItem_T *)alloc(len * sizeof(sortItem_T));
if (ptrs == NULL)
goto theend;
*/
++p;
len = getdigits(&p);
- buf = alloc((int)(len + 1));
+ buf = alloc(len + 1);
if (buf == NULL)
return TRUE;
p = buf;
oldlen = STRLEN(p_rtp);
addlen = STRLEN(fname) + 1; // add one for comma
- new_rtp = alloc((int)(oldlen + addlen + afterlen + 1)); // add one for NUL
+ new_rtp = alloc(oldlen + addlen + afterlen + 1); // add one for NUL
if (new_rtp == NULL)
goto theend;
while ((pos = (char_u *)strstr((char *)pos + 2, "$*")) != NULL)
++i;
len = (int)STRLEN(p);
- new_cmdline = alloc((int)(STRLEN(program) + i * (len - 2) + 1));
+ new_cmdline = alloc(STRLEN(program) + i * (len - 2) + 1);
if (new_cmdline == NULL)
return NULL; /* out of memory */
ptr = new_cmdline;
}
else
{
- new_cmdline = alloc((int)(STRLEN(program) + STRLEN(p) + 2));
+ new_cmdline = alloc(STRLEN(program) + STRLEN(p) + 2);
if (new_cmdline == NULL)
return NULL; /* out of memory */
STRCPY(new_cmdline, program);
{
if (newlen)
{
- temp = (histentry_T *)alloc(
- (long_u)(newlen * sizeof(histentry_T)));
+ temp = (histentry_T *)alloc(newlen * sizeof(histentry_T));
if (temp == NULL) /* out of memory! */
{
if (type == 0) /* first one: just keep the old length */
viminfo_history[type] = NULL;
else
viminfo_history[type] = (histentry_T *)lalloc(
- (long_u)(len * sizeof(histentry_T)), FALSE);
+ len * sizeof(histentry_T), FALSE);
if (viminfo_history[type] == NULL)
len = 0;
viminfo_hislen[type] = len;
ga_init2(gap, (int)sizeof(char *), 20);
#ifdef MSWIN
- buf = alloc((int)MAXPATHL);
+ buf = alloc(MAXPATHL);
if (buf == NULL)
return FAIL;
STRNCPY(buf, path, MAXPATHL-5);
if (search_ctx->ffsc_wc_path != NULL)
{
wc_path = vim_strsave(search_ctx->ffsc_wc_path);
- temp = alloc((int)(STRLEN(search_ctx->ffsc_wc_path)
+ temp = alloc(STRLEN(search_ctx->ffsc_wc_path)
+ STRLEN(search_ctx->ffsc_fix_path + len)
- + 1));
+ + 1);
if (temp == NULL || wc_path == NULL)
{
vim_free(buf);
* filepath is used as buffer for various actions and as the storage to
* return a found filename.
*/
- if ((file_path = alloc((int)MAXPATHL)) == NULL)
+ if ((file_path = alloc(MAXPATHL)) == NULL)
return NULL;
#ifdef FEAT_PATH_EXTRA
break;
}
- if ((buf = alloc((int)(MAXPATHL))) == NULL)
+ if ((buf = alloc(MAXPATHL)) == NULL)
break;
// copy next path
char_u *p;
int len;
- if ((buf = alloc((int)MAXPATHL)) == NULL)
+ if ((buf = alloc(MAXPATHL)) == NULL)
return;
while (*path_option != NUL)
if (regmatch.regprog == NULL)
return;
- if ((curdir = alloc((int)(MAXPATHL))) == NULL)
+ if ((curdir = alloc(MAXPATHL)) == NULL)
goto theend;
mch_dirname(curdir, MAXPATHL);
expand_path_option(curdir, &path_ga);
continue;
}
- rel_path = alloc((int)(STRLEN(short_name) + STRLEN(PATHSEPSTR) + 2));
+ rel_path = alloc(STRLEN(short_name) + STRLEN(PATHSEPSTR) + 2);
if (rel_path == NULL)
goto theend;
STRCPY(rel_path, ".");
if (len == 0)
return E_INVALIDARG;
- buffer = (char *)alloc((unsigned)len);
+ buffer = (char *)alloc(len);
if (buffer == NULL)
return E_OUTOFMEMORY;
char_u *np;
size_t len = STRLEN(p) + 1;
- if ((np = alloc((int)len + 2)) == NULL)
+ if ((np = alloc(len + 2)) == NULL)
{
vim_free(p);
return NULL;
if (exported)
{
pt->pt_argv = (typval_T *)alloc_clear(
- sizeof(typval_T) * self->argc);
+ sizeof(typval_T) * self->argc);
for (i = 0; i < pt->pt_argc; ++i)
copy_tv(&self->argv[i], &pt->pt_argv[i]);
}
/* Create a copy of the string, with internal nulls replaced by
* newline characters, as is the vim convention.
*/
- save = (char *)alloc((unsigned)(len+1));
+ save = (char *)alloc(len+1);
if (save == NULL)
{
PyErr_NoMemory();
list_alloc_id(alloc_id_T id UNUSED)
{
#ifdef FEAT_EVAL
- if (alloc_fail_id == id && alloc_does_fail((long_u)sizeof(list_T)))
+ if (alloc_fail_id == id && alloc_does_fail(sizeof(list_T)))
return NULL;
#endif
return (list_alloc());
rettv_list_alloc_id(typval_T *rettv, alloc_id_T id UNUSED)
{
#ifdef FEAT_EVAL
- if (alloc_fail_id == id && alloc_does_fail((long_u)sizeof(list_T)))
+ if (alloc_fail_id == id && alloc_does_fail(sizeof(list_T)))
return FAIL;
#endif
return rettv_list_alloc(rettv);
(void)delete_first_msg();
/* allocate an entry and add the message at the end of the history */
- p = (struct msg_hist *)alloc((int)sizeof(struct msg_hist));
+ p = (struct msg_hist *)alloc(sizeof(struct msg_hist));
if (p != NULL)
{
if (len < 0)
if (s > *sb_str)
{
- mp = (msgchunk_T *)alloc((int)(sizeof(msgchunk_T) + (s - *sb_str)));
+ mp = (msgchunk_T *)alloc(sizeof(msgchunk_T) + (s - *sb_str));
if (mp != NULL)
{
mp->sb_eol = finish;
// Make room for file name. When doing encoding conversion the actual
// length may be quite a bit longer, thus use the maximum possible length.
- buf = alloc((int)MAXPATHL);
+ buf = alloc(MAXPATHL);
if (buf == NULL)
return 0;
}
/* make room for file name */
- buf = alloc((int)STRLEN(path) + BASENAMELEN + 5);
+ buf = alloc(STRLEN(path) + BASENAMELEN + 5);
if (buf == NULL)
return 0;
{
char_u *p;
- p = alloc((size_t)(len + 1));
+ p = alloc(len + 1);
if (p != NULL)
{
STRNCPY(p, string, len);
free_yank_all();
*y_current = *curr;
y_current->y_array = (char_u **)lalloc_clear(
- (long_u)(sizeof(char_u *) * y_current->y_size), TRUE);
+ sizeof(char_u *) * y_current->y_size, TRUE);
if (y_current->y_array == NULL)
y_current->y_size = 0;
else
else if ((sbuf = getenv((char *)lognam)))
{
lengte = strlen(sbuf) + 1;
- cp = (char_u *)alloc((size_t)lengte);
+ cp = (char_u *)alloc(lengte);
if (cp)
strcpy((char *)cp, sbuf);
return cp;
return;
/* Remember the buffer numbers for the arguments. */
- fnum_list = (int *)alloc((int)sizeof(int) * GARGCOUNT);
+ fnum_list = (int *)alloc(sizeof(int) * GARGCOUNT);
if (fnum_list == NULL)
return; /* out of memory */
for (i = 0; i < GARGCOUNT; ++i)
if (title != NULL)
{
- char_u *p = alloc((int)STRLEN(title) + 2);
+ char_u *p = alloc(STRLEN(title) + 2);
qfl->qf_title = p;
if (p != NULL)
emsg(_(e_maxmempat));
return NULL;
}
- newl = (nfa_thread_T *)alloc((int)newsize);
+ newl = (nfa_thread_T *)alloc(newsize);
if (newl == NULL)
return NULL;
l->len = newlen;
/* Allocate space to save the text displayed in the command line area. */
rows = screen_Rows - cmdline_row;
- screenline = (schar_T *)lalloc(
- (long_u)(rows * cols * sizeof(schar_T)), FALSE);
- screenattr = (sattr_T *)lalloc(
- (long_u)(rows * cols * sizeof(sattr_T)), FALSE);
+ screenline = (schar_T *)lalloc(rows * cols * sizeof(schar_T), FALSE);
+ screenattr = (sattr_T *)lalloc(rows * cols * sizeof(sattr_T), FALSE);
if (screenline == NULL || screenattr == NULL)
ret = 2;
if (enc_utf8)
{
screenlineUC = (u8char_T *)lalloc(
- (long_u)(rows * cols * sizeof(u8char_T)), FALSE);
+ rows * cols * sizeof(u8char_T), FALSE);
if (screenlineUC == NULL)
ret = 2;
for (i = 0; i < p_mco; ++i)
{
screenlineC[i] = (u8char_T *)lalloc(
- (long_u)(rows * cols * sizeof(u8char_T)), FALSE);
+ rows * cols * sizeof(u8char_T), FALSE);
if (screenlineC[i] == NULL)
ret = 2;
}
}
if (enc_dbcs == DBCS_JPNU)
{
- screenline2 = (schar_T *)lalloc(
- (long_u)(rows * cols * sizeof(schar_T)), FALSE);
+ screenline2 = (schar_T *)lalloc(rows * cols * sizeof(schar_T), FALSE);
if (screenline2 == NULL)
ret = 2;
}
// Reserve enough space for the search pattern + offset.
len = STRLEN(p) + off_len + 3;
- msgbuf = alloc((int)len);
+ msgbuf = alloc(len);
if (msgbuf != NULL)
{
vim_memset(msgbuf, ' ', len);
if (HASHITEM_EMPTY(hi))
{
// new group
- group = (signgroup_T *)alloc(
- (unsigned)(sizeof(signgroup_T) + STRLEN(groupname)));
+ group = (signgroup_T *)alloc(sizeof(signgroup_T) + STRLEN(groupname));
if (group == NULL)
return NULL;
STRCPY(group->sg_name, groupname);
int start = next_sign_typenr;
// Allocate a new sign.
- sp = (sign_T *)alloc_clear_id((unsigned)sizeof(sign_T),
- aid_sign_define_by_name);
+ sp = (sign_T *)alloc_clear_id(sizeof(sign_T), aid_sign_define_by_name);
if (sp == NULL)
return NULL;
name_folded, MAXKEYWLEN + 1);
else
name_ic = name;
- kp = (keyentry_T *)alloc((int)(sizeof(keyentry_T) + STRLEN(name_ic)));
+ kp = (keyentry_T *)alloc(sizeof(keyentry_T) + STRLEN(name_ic));
if (kp == NULL)
return;
STRCPY(kp->keyword, name_ic);
{
for (end = p; *end && !VIM_ISWHITE(*end) && *end != ','; ++end)
;
- name = alloc((int)(end - p + 3)); /* leave room for "^$" */
+ name = alloc(end - p + 3); /* leave room for "^$" */
if (name == NULL)
{
failed = TRUE;
if (name_only)
mfp = vim_strsave(res_name);
else
- mfp = (char_u *)alloc((int)sizeof(char_u) + len + 1);
+ mfp = (char_u *)alloc(sizeof(char_u) + len + 1);
if (mfp == NULL)
continue;
*/
*tagp.tagname_end = NUL;
len = (int)(tagp.tagname_end - tagp.tagname);
- mfp = (char_u *)alloc((int)sizeof(char_u)
+ mfp = (char_u *)alloc(sizeof(char_u)
+ len + 10 + ML_EXTRA + 1);
if (mfp != NULL)
{
else
{
len = (int)(tagp.tagname_end - tagp.tagname);
- mfp = (char_u *)alloc((int)sizeof(char_u) + len + 1);
+ mfp = (char_u *)alloc(sizeof(char_u) + len + 1);
if (mfp != NULL)
vim_strncpy(mfp, tagp.tagname, len);
else
++len;
#endif
- mfp = (char_u *)alloc((int)sizeof(char_u) + len + 1);
+ mfp = (char_u *)alloc(sizeof(char_u) + len + 1);
if (mfp != NULL)
{
p = mfp;
/* Make a copy of the line, it can become invalid when an autocommand calls
* back here recursively. */
len = matching_line_len(lbuf_arg) + 1;
- lbuf = alloc((int)len);
+ lbuf = alloc(len);
if (lbuf != NULL)
mch_memmove(lbuf, lbuf_arg, len);
{
tc_max_len += 20;
new_tc = (struct termcode *)alloc(
- (unsigned)(tc_max_len * sizeof(struct termcode)));
+ tc_max_len * sizeof(struct termcode));
if (new_tc == NULL)
{
tc_max_len -= 20;
if (!counting)
{
colornames_table = (struct rgbcolor_table_S *)alloc(
- (unsigned)(sizeof(struct rgbcolor_table_S) * size));
+ sizeof(struct rgbcolor_table_S) * size);
if (colornames_table == NULL)
{
fclose(fd);
cmd = (char_u*)"";
len = STRLEN(cmd) + 10;
- p = alloc((int)len);
+ p = alloc(len);
for (i = 0; p != NULL; ++i)
{
if (len == 0)
p = NULL;
else
- p = (cellattr_T *)alloc((int)sizeof(cellattr_T) * len);
+ p = (cellattr_T *)alloc(sizeof(cellattr_T) * len);
if ((p != NULL || len == 0)
&& ga_grow(&term->tl_scrollback, 1) == OK)
{
ga_init2(&ga, 1, 100);
if (len > 0)
- p = (cellattr_T *)alloc((int)sizeof(cellattr_T) * len);
+ p = (cellattr_T *)alloc(sizeof(cellattr_T) * len);
if (p != NULL)
{
for (col = 0; col < len; col += cells[col].width)
p = dict_get_string(dict, (char_u *)"encoding", FALSE);
if (p != NULL)
{
- ea.cmd = alloc((int)STRLEN(p) + 12);
+ ea.cmd = alloc(STRLEN(p) + 12);
if (ea.cmd != NULL)
{
sprintf((char *)ea.cmd, "sbuf ++enc=%s", p);
else
txt = (char_u *)_("finished");
len = 9 + STRLEN(term->tl_buffer->b_fname) + STRLEN(txt);
- term->tl_status_text = alloc((int)len);
+ term->tl_status_text = alloc(len);
if (term->tl_status_text != NULL)
vim_snprintf((char *)term->tl_status_text, len, "%s [%s]",
term->tl_buffer->b_fname, txt);
{
size_t len = STRLEN(fname1) + 12;
- fname_tofree = alloc((int)len);
+ fname_tofree = alloc(len);
if (fname_tofree != NULL)
{
vim_snprintf((char *)fname_tofree, len, "dump diff %s", fname1);
else
{
size_t size = sizeof(sb_line_T) * term->tl_scrollback.ga_len;
- sb_line_T *temp = (sb_line_T *)alloc((int)size);
+ sb_line_T *temp = (sb_line_T *)alloc(size);
/* need to copy cell properties into temp memory */
if (temp != NULL)
{
/* Request by CreateProcessW */
breq = wcslen(cmd_wchar) + 1 + 1; /* Addition of NUL by API */
- cmd_wchar_copy = (PWSTR)alloc((int)(breq * sizeof(WCHAR)));
+ cmd_wchar_copy = (PWSTR)alloc(breq * sizeof(WCHAR));
wcsncpy(cmd_wchar_copy, cmd_wchar, breq - 1);
}
/* Set up pipe inheritance safely: Vista or later. */
pInitializeProcThreadAttributeList(NULL, 1, 0, &breq);
term->tl_siex.lpAttributeList =
- (PPROC_THREAD_ATTRIBUTE_LIST)alloc((int)breq);
+ (PPROC_THREAD_ATTRIBUTE_LIST)alloc(breq);
if (!term->tl_siex.lpAttributeList)
goto failed;
if (!pInitializeProcThreadAttributeList(term->tl_siex.lpAttributeList, 1,
oldproplen = get_text_props(curbuf, lnum, &props, FALSE);
len = STRLEN(newp) + 1;
- line = alloc((int)(len + (oldproplen + proplen) * sizeof(textprop_T)));
+ line = alloc(len + (oldproplen + proplen) * sizeof(textprop_T));
if (line == NULL)
return;
mch_memmove(line, newp, len);
static int included_patches[] =
{ /* Add new patch number below this line */
+/**/
+ 1393,
/**/
1392,
/**/