aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-09-27 04:14:06 -04:00
Code Activity

patch 8.0.1723: using one item array size declaration is misleading

Browse Source 
Problem:    Using one item array size declaration is misleading.
Solution:   Instead of using "[1]" and actually using a larger array, use
            "[]".  This is to verify that this C99 feature works for all
            compilers.
This commit is contained in:
Bram Moolenaar
2018-04-16 14:45:45 +02:00
parent 498c2562e1
commit f3a411783c
3 changed files with 53 additions and 41 deletions
Download Patch File Download Diff File Expand all files Collapse all files

76
src/getchar.c
View File

@@ -40,9 +40,9 @@
#define MINIMAL_SIZE 20 /* minimal size for b_str */ #define MINIMAL_SIZE 20 /* minimal size for b_str */
static buffheader_T redobuff = {{NULL, {NUL}}, NULL, 0, 0}; static buffheader_T redobuff = {NULL, NULL, 0, 0};
static buffheader_T old_redobuff = {{NULL, {NUL}}, NULL, 0, 0}; static buffheader_T old_redobuff = {NULL, NULL, 0, 0};
static buffheader_T recordbuff = {{NULL, {NUL}}, NULL, 0, 0}; static buffheader_T recordbuff = {NULL, NULL, 0, 0};
static int typeahead_char = 0; /* typeahead char that's not flushed */ static int typeahead_char = 0; /* typeahead char that's not flushed */
@@ -138,12 +138,13 @@ free_buff(buffheader_T *buf)
{ {
buffblock_T *p, *np; buffblock_T *p, *np;
for (p = buf->bh_first.b_next; p != NULL; p = np) for (p = buf->bh_first; p != NULL; p = np)
{ {
np = p->b_next; np = p->b_next;
vim_free(p); vim_free(p);
} }
buf->bh_first.b_next = NULL; buf->bh_first = NULL;
buf->bh_curr = NULL;
} }
/* /*
@@ -162,13 +163,13 @@ get_buffcont(
buffblock_T *bp; buffblock_T *bp;
/* compute the total length of the string */ /* compute the total length of the string */
for (bp = buffer->bh_first.b_next; bp != NULL; bp = bp->b_next) for (bp = buffer->bh_first; bp != NULL; bp = bp->b_next)
count += (long_u)STRLEN(bp->b_str); count += (long_u)STRLEN(bp->b_str);
if ((count || dozero) && (p = lalloc(count + 1, TRUE)) != NULL) if ((count || dozero) && (p = lalloc(count + 1, TRUE)) != NULL)
{ {
p2 = p; p2 = p;
for (bp = buffer->bh_first.b_next; bp != NULL; bp = bp->b_next) for (bp = buffer->bh_first; bp != NULL; bp = bp->b_next)
for (str = bp->b_str; *str; ) for (str = bp->b_str; *str; )
*p2++ = *str++; *p2++ = *str++;
*p2 = NUL; *p2 = NUL;
@@ -239,10 +240,10 @@ add_buff(
if (slen == 0) /* don't add empty strings */ if (slen == 0) /* don't add empty strings */
return; return;
if (buf->bh_first.b_next == NULL) /* first add to list */ if (buf->bh_first == NULL) /* first add to list */
{ {
buf->bh_space = 0; buf->bh_space = 0;
buf->bh_curr = &(buf->bh_first); buf->bh_curr = NULL;
} }
else if (buf->bh_curr == NULL) /* buffer has already been read */ else if (buf->bh_curr == NULL) /* buffer has already been read */
{ {
@@ -250,9 +251,9 @@ add_buff(
return; return;
} }
else if (buf->bh_index != 0) else if (buf->bh_index != 0)
mch_memmove(buf->bh_first.b_next->b_str, mch_memmove(buf->bh_first->b_str,
buf->bh_first.b_next->b_str + buf->bh_index, buf->bh_first->b_str + buf->bh_index,
STRLEN(buf->bh_first.b_next->b_str + buf->bh_index) + 1); STRLEN(buf->bh_first->b_str + buf->bh_index) + 1);
buf->bh_index = 0; buf->bh_index = 0;
if (buf->bh_space >= (int)slen) if (buf->bh_space >= (int)slen)
@@ -267,17 +268,26 @@ add_buff(
len = MINIMAL_SIZE; len = MINIMAL_SIZE;
else else
len = slen; len = slen;
p = (buffblock_T *)lalloc((long_u)(sizeof(buffblock_T) + len), p = (buffblock_T *)lalloc((long_u)(sizeof(buffblock_T) + len + 1),
TRUE); TRUE);
if (p == NULL) if (p == NULL)
return; /* no space, just forget it */ return; /* no space, just forget it */
buf->bh_space = (int)(len - slen); buf->bh_space = (int)(len - slen);
vim_strncpy(p->b_str, s, (size_t)slen); vim_strncpy(p->b_str, s, (size_t)slen);
if (buf->bh_curr == NULL)
{
p->b_next = NULL;
buf->bh_first = p;
buf->bh_curr = p;
}
else
{
p->b_next = buf->bh_curr->b_next; p->b_next = buf->bh_curr->b_next;
buf->bh_curr->b_next = p; buf->bh_curr->b_next = p;
buf->bh_curr = p; buf->bh_curr = p;
} }
}
return; return;
} }
@@ -348,10 +358,10 @@ add_char_buff(buffheader_T *buf, int c)
} }
/* First read ahead buffer. Used for translated commands. */ /* First read ahead buffer. Used for translated commands. */
static buffheader_T readbuf1 = {{NULL, {NUL}}, NULL, 0, 0}; static buffheader_T readbuf1 = {NULL, NULL, 0, 0};
/* Second read ahead buffer. Used for redo. */ /* Second read ahead buffer. Used for redo. */
static buffheader_T readbuf2 = {{NULL, {NUL}}, NULL, 0, 0}; static buffheader_T readbuf2 = {NULL, NULL, 0, 0};
/* /*
* Get one byte from the read buffers. Use readbuf1 one first, use readbuf2 * Get one byte from the read buffers. Use readbuf1 one first, use readbuf2
@@ -376,17 +386,17 @@ read_readbuf(buffheader_T *buf, int advance)
char_u c; char_u c;
buffblock_T *curr; buffblock_T *curr;
if (buf->bh_first.b_next == NULL) /* buffer is empty */ if (buf->bh_first == NULL) /* buffer is empty */
return NUL; return NUL;
curr = buf->bh_first.b_next; curr = buf->bh_first;
c = curr->b_str[buf->bh_index]; c = curr->b_str[buf->bh_index];
if (advance) if (advance)
{ {
if (curr->b_str[++buf->bh_index] == NUL) if (curr->b_str[++buf->bh_index] == NUL)
{ {
buf->bh_first.b_next = curr->b_next; buf->bh_first = curr->b_next;
vim_free(curr); vim_free(curr);
buf->bh_index = 0; buf->bh_index = 0;
} }
@@ -400,14 +410,14 @@ read_readbuf(buffheader_T *buf, int advance)
static void static void
start_stuff(void) start_stuff(void)
{ {
if (readbuf1.bh_first.b_next != NULL) if (readbuf1.bh_first != NULL)
{ {
readbuf1.bh_curr = &(readbuf1.bh_first); readbuf1.bh_curr = readbuf1.bh_first;
readbuf1.bh_space = 0; readbuf1.bh_space = 0;
} }
if (readbuf2.bh_first.b_next != NULL) if (readbuf2.bh_first != NULL)
{ {
readbuf2.bh_curr = &(readbuf2.bh_first); readbuf2.bh_curr = readbuf2.bh_first;
readbuf2.bh_space = 0; readbuf2.bh_space = 0;
} }
} }
@@ -418,8 +428,8 @@ start_stuff(void)
int int
stuff_empty(void) stuff_empty(void)
{ {
return (readbuf1.bh_first.b_next == NULL return (readbuf1.bh_first == NULL
&& readbuf2.bh_first.b_next == NULL); && readbuf2.bh_first == NULL);
} }
/* /*
@@ -429,7 +439,7 @@ stuff_empty(void)
int int
readbuf1_empty(void) readbuf1_empty(void)
{ {
return (readbuf1.bh_first.b_next == NULL); return (readbuf1.bh_first == NULL);
} }
/* /*
@@ -494,7 +504,7 @@ ResetRedobuff(void)
{ {
free_buff(&old_redobuff); free_buff(&old_redobuff);
old_redobuff = redobuff; old_redobuff = redobuff;
redobuff.bh_first.b_next = NULL; redobuff.bh_first = NULL;
} }
} }
@@ -509,7 +519,7 @@ CancelRedo(void)
{ {
free_buff(&redobuff); free_buff(&redobuff);
redobuff = old_redobuff; redobuff = old_redobuff;
old_redobuff.bh_first.b_next = NULL; old_redobuff.bh_first = NULL;
start_stuff(); start_stuff();
while (read_readbuffers(TRUE) != NUL) while (read_readbuffers(TRUE) != NUL)
; ;
@@ -526,9 +536,9 @@ saveRedobuff(save_redo_T *save_redo)
char_u *s; char_u *s;
save_redo->sr_redobuff = redobuff; save_redo->sr_redobuff = redobuff;
redobuff.bh_first.b_next = NULL; redobuff.bh_first = NULL;
save_redo->sr_old_redobuff = old_redobuff; save_redo->sr_old_redobuff = old_redobuff;
old_redobuff.bh_first.b_next = NULL; old_redobuff.bh_first = NULL;
/* Make a copy, so that ":normal ." in a function works. */ /* Make a copy, so that ":normal ." in a function works. */
s = get_buffcont(&save_redo->sr_redobuff, FALSE); s = get_buffcont(&save_redo->sr_redobuff, FALSE);
@@ -747,9 +757,9 @@ read_redo(int init, int old_redo)
if (init) if (init)
{ {
if (old_redo) if (old_redo)
bp = old_redobuff.bh_first.b_next; bp = old_redobuff.bh_first;
else else
bp = redobuff.bh_first.b_next; bp = redobuff.bh_first;
if (bp == NULL) if (bp == NULL)
return FAIL; return FAIL;
p = bp->b_str; p = bp->b_str;
@@ -1372,9 +1382,9 @@ save_typeahead(tasave_T *tp)
old_char = -1; old_char = -1;
tp->save_readbuf1 = readbuf1; tp->save_readbuf1 = readbuf1;
readbuf1.bh_first.b_next = NULL; readbuf1.bh_first = NULL;
tp->save_readbuf2 = readbuf2; tp->save_readbuf2 = readbuf2;
readbuf2.bh_first.b_next = NULL; readbuf2.bh_first = NULL;
# ifdef USE_INPUT_BUF # ifdef USE_INPUT_BUF
tp->save_inputbuf = get_input_buf(); tp->save_inputbuf = get_input_buf();
# endif # endif

4
src/structs.h
View File

@@ -511,7 +511,7 @@ typedef struct buffheader buffheader_T;
struct buffblock struct buffblock
{ {
buffblock_T *b_next; /* pointer to next buffblock */ buffblock_T *b_next; /* pointer to next buffblock */
char_u b_str[1]; /* contents (actually longer) */ char_u b_str[]; /* contents (flexible array) */
}; };
/* /*
@@ -519,7 +519,7 @@ struct buffblock
*/ */
struct buffheader struct buffheader
{ {
buffblock_T bh_first; /* first (dummy) block of list */ buffblock_T *bh_first; /* first block of the list */
buffblock_T *bh_curr; /* buffblock for appending */ buffblock_T *bh_curr; /* buffblock for appending */
int bh_index; /* index for reading */ int bh_index; /* index for reading */
int bh_space; /* space in bh_curr for appending */ int bh_space; /* space in bh_curr for appending */

2
src/version.c
View File

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