re

Author: codingforkind

mirror-common/src/main/java/cn/com/mirror/analyser/visitor/VariableVisitor.java

@@ -97,9 +97,6 @@ public boolean visit(MethodDeclaration node) {
 
     @Override
     public boolean visit(SimpleName node) {
-        // TODO xyz the statement of "return this;" and "return null" can not be visited
-        // return statement directly control dependence on the method it belongs.
-
         IBinding binding = node.resolveBinding();
         if (binding instanceof IVariableBinding) {
             IVariableBinding iVariableBinding = (IVariableBinding) binding;
@@ -116,11 +113,6 @@ public boolean visit(SimpleName node) {
     private void addVariable(Integer lineNum,
                              Variable variable) {
 
-        if ("/home/piggy/work/mirror/mirror-common/src/main/java/cn/com/mirror/project/unit/element/Method.java"
-                .equals(variable.getFile()) && (lineNum == 49 || lineNum == 51)) {
-            log.debug("a;lsjd;lfjsajdlfjalsjd;lf");
-        }
-
         if (!varStatMap.containsKey(lineNum)) {
             Statement statement = new Statement(this.file,
                     lineNum,

mirror-common/src/main/java/cn/com/mirror/reflect/EdgeConstructor.java

@@ -60,6 +60,10 @@ public void construct() {
                 Base tailB = getBaseElement(ctrlVal);
 
                 if (null == headB || null == tailB) {
+                    // TODO xyz the statement of "return this;" and "return null" can not be visited,
+                    // directly result in that head statement can not be retrieve from the base elements.
+                    // do some spacial treatment for "return this;" and "return null".
+
                     log.debug("TARGET: {}", targetPath);
                     log.debug("HEAD: {}:{} \t->\t TAIL: {}:{}", ctrlKey.getLineNum(),
                             ctrlKey.getVertexType(), ctrlVal.getLineNum(), ctrlVal.getVertexType());