patch 8.1.1414: alloc() returning "char_u *" causes a lot of type casts
Problem: Alloc() returning "char_u *" causes a lot of type casts.
Solution: Have it return "void *". (Mike Williams) Define ALLOC_ONE() to
check the simple allocations.
diff --git a/src/quickfix.c b/src/quickfix.c
index 4f3db39..e237df0 100644
--- a/src/quickfix.c
+++ b/src/quickfix.c
@@ -540,7 +540,7 @@
while (efm[0] != NUL)
{
// Allocate a new eformat structure and put it at the end of the list
- fmt_ptr = (efm_T *)alloc_clear(sizeof(efm_T));
+ fmt_ptr = ALLOC_CLEAR_ONE(efm_T);
if (fmt_ptr == NULL)
goto parse_efm_error;
if (fmt_first == NULL) // first one
@@ -1890,7 +1890,7 @@
{
qf_delq_T *q;
- q = (qf_delq_T *)alloc(sizeof(qf_delq_T));
+ q = ALLOC_ONE(qf_delq_T);
if (q != NULL)
{
q->qi = qi;
@@ -2063,7 +2063,7 @@
qfline_T *qfp;
qfline_T **lastp; // pointer to qf_last or NULL
- if ((qfp = (qfline_T *)alloc(sizeof(qfline_T))) == NULL)
+ if ((qfp = ALLOC_ONE(qfline_T)) == NULL)
return QF_FAIL;
if (bufnum != 0)
{
@@ -2141,7 +2141,7 @@
{
qf_info_T *qi;
- qi = (qf_info_T *)alloc_clear(sizeof(qf_info_T));
+ qi = ALLOC_CLEAR_ONE(qf_info_T);
if (qi != NULL)
{
qi->qf_refcount++;
@@ -2429,7 +2429,7 @@
struct dir_stack_T *ds_ptr;
// allocate new stack element and hook it in
- ds_new = (struct dir_stack_T *)alloc(sizeof(struct dir_stack_T));
+ ds_new = ALLOC_ONE(struct dir_stack_T);
if (ds_new == NULL)
return NULL;