[Cpptool-develop] ASTNodeTypeFeatures and ASTNodePropertyFeatures...

[Baptiste L. <gai...@fr...>]

    I'm not really happy with the ASTNodeTypeFeatures and
ASTNodePropertyFeatures  features. They seems to only be used to 'optimize'
the visit (mayHave...). In practice, this means that each time we add a
visitor for a specific kind of entity, some features are added to speed up
the visit. This sound plain wrong to me (you have to change most of the
ASTNodeTypes construction to do that).

    Also, doing it that way make it difficult to see what is being visited
as you have to search through all the node types for some specific features.

    I'm thinking of removing feature support for ASTNodeType and
ASTNodeProperty (or a least remove its usage for driving visitation).
ASTNodePropertyFeatures is in fact unused. An alternative to drive the node
visitation would probably be to declare what really need to be visited in
the visitor, much like what has been done in the IdentifierResolver.

What do you think ?

Baptiste.