bitq: remove a static variable

in_buff is only ever set to field->in_value and that pointer is still
available when the parsing is restarted so it could just as well be used
directly, removing the need for the static variable.

Change-Id: I3dd7a8315ed5c5bdc3bfb74044f89492bca9816c
Signed-off-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Reviewed-on: http://openocd.zylin.com/206
Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
This commit is contained in:
Andreas Fritiofson 2011-11-15 23:56:04 +01:00 committed by Spencer Oliver
parent d31803d3d9
commit 9450da873c
1 changed files with 2 additions and 4 deletions

View File

@ -44,7 +44,6 @@ static struct bitq_state bitq_in_state;
void bitq_in_proc(void)
{
/* static information preserved between calls to increase performance */
static uint8_t* in_buff; /* pointer to buffer for scanned data */
static int in_idx; /* index of byte being scanned */
static uint8_t in_mask; /* mask of next bit to be scanned */
@ -68,7 +67,6 @@ void bitq_in_proc(void)
/* initialize field scanning */
in_mask = 0x01;
in_idx = 0;
in_buff = field->in_value;
}
/* field scanning */
@ -82,9 +80,9 @@ void bitq_in_proc(void)
return;
}
if (in_mask == 0x01)
in_buff[in_idx] = 0;
field->in_value[in_idx] = 0;
if (tdo)
in_buff[in_idx] |= in_mask;
field->in_value[in_idx] |= in_mask;
if (in_mask == 0x80)
{
in_mask = 0x01;