walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

Addons; split up possible/known values and impossible values. This is inconsistent with core Cppcheck, however the addons do not handle impossible values in general. A future improvement might be to clarify this somehow, maybe renaming Token.values.

This commit is contained in:
Daniel Marjamäki 2021-02-23 22:40:49 +01:00
parent fc1cfba5e0
commit 22727ee3ab
1 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
addons/cppcheckdata.py
View File

@ -145,7 +145,8 @@ class Token:
variable Variable information for this token. See the Variable class. variable Variable information for this token. See the Variable class.
function If this token points at a function call, this attribute has the Function function If this token points at a function call, this attribute has the Function
information. See the Function class. information. See the Function class.
values Possible values of token values Possible/Known values of token
impossible_values Impossible values of token
valueType type information valueType type information
typeScope type scope (token->type()->classScope) typeScope type scope (token->type()->classScope)
astParent ast parent astParent ast parent
@ -198,6 +199,7 @@ class Token:
function = None function = None
valuesId = None valuesId = None
values = None values = None
impossible_values = None
valueType = None valueType = None
typeScopeId = None typeScopeId = None
@ -299,7 +301,14 @@ class Token:
self.link = IdMap[self.linkId] self.link = IdMap[self.linkId]
self.variable = IdMap[self.variableId] self.variable = IdMap[self.variableId]
self.function = IdMap[self.functionId] self.function = IdMap[self.functionId]
self.values = IdMap[self.valuesId] self.values = []
self.impossible_values = []
if IdMap[self.valuesId]:
for v in IdMap[self.valuesId]:
if v.isImpossible():
self.impossible_values.append(v)
else:
self.values.append(v)
self.typeScope = IdMap[self.typeScopeId] self.typeScope = IdMap[self.typeScopeId]
self.astParent = IdMap[self.astParentId] self.astParent = IdMap[self.astParentId]
self.astOperand1 = IdMap[self.astOperand1Id] self.astOperand1 = IdMap[self.astOperand1Id]
@ -571,6 +580,9 @@ class Value:
def isPossible(self): def isPossible(self):
return self.valueKind and self.valueKind == 'possible' return self.valueKind and self.valueKind == 'possible'
def isImpossible(self):
return self.valueKind and self.valueKind == 'impossible'
def __init__(self, element): def __init__(self, element):
self.intvalue = element.get('intvalue') self.intvalue = element.get('intvalue')
if self.intvalue: if self.intvalue:
@ -585,6 +597,8 @@ class Value:
self.valueKind = 'known' self.valueKind = 'known'
elif element.get('possible'): elif element.get('possible'):
self.valueKind = 'possible' self.valueKind = 'possible'
elif element.get('impossible'):
self.valueKind = 'impossible'
if element.get('inconclusive'): if element.get('inconclusive'):
self.inconclusive = True self.inconclusive = True