diff options
author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2013-12-26 09:40:00 -0800 |
---|---|---|
committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2013-12-26 09:40:00 -0800 |
commit | 0037a42107b952c9d903719615747e760e4e7247 (patch) | |
tree | 261a9255c69f176d1d295eb7604ca95a06ff20e7 | |
parent | 3acba630d8b57084f7e92c15732408711ed5137a (diff) |
Initialize both value1 & value2, not value1 twice
Flagged by cppcheck 1.62:
[src/encparse.c:303] -> [src/encparse.c:303]: (performance, inconclusive)
Variable 'value1' is reassigned a value before the old one has been used
if variable is no semaphore variable.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
-rw-r--r-- | src/encparse.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/encparse.c b/src/encparse.c index ee18b3f..d10634d 100644 --- a/src/encparse.c +++ b/src/encparse.c @@ -300,7 +300,7 @@ getnextline(FontFilePtr f) } } else if(!strcasecmp(keyword_value, "STARTMAPPING")) { keyword_value[0] = 0; - value1 = 0; value1 = 0; + value1 = 0; value2 = 0; /* first a keyword */ token = gettoken(f,c,&c); if(token != KEYWORD_TOKEN) { |