CodeCompass icon indicating copy to clipboard operation
CodeCompass copied to clipboard

CppAstNode astType usage should be something better

Open bruntib opened this issue 7 years ago • 2 comments

The astType of a CppAstNode can be "usage". We use this for several reference types like function call, inheritance, etc. Instead of "usage" we should introduce "call", "inheritance", etc. Not to mention that we also have an astType for virtual function call, and the variable usages are also "read" or "write".

bruntib avatar Nov 17 '18 16:11 bruntib

There is also a TypeLocation which should be eliminated. We should use the manner of usage instead, like ParameterTypeLoc, GlobalTypeLoc, etc.

bruntib avatar Dec 08 '18 13:12 bruntib

A caught exception object is currently considered a local variable. A new type location type should be introduced on this:

try { ... }
catch (const MyType&) { ... }

bruntib avatar Dec 08 '18 13:12 bruntib