mauricioaniche/ck

Add all endVisit() callbacks in CKVisitor

mauricioaniche opened this issue · 1 comments

CKVisitor is the one that JDT invokes whenever it visits a Node. CKVisitor then calls the visits inside each of the metrics. So, CKVisitor should propagate all calls to visitX() or endVisitX(), in the metrics.

It's a boring copy-and-paste task, but we need to add all of them for the endVisits() as we still miss some. I have been adding them on-demand (i.e., whenever a metric needs it, I add it). Of course, it always takes me 15 minutes to notice that a bug that is caused simply because the method is not being called.

I've made the pull request to this problem