tweak

git-svn-id: https://pmd.svn.sourceforge.net/svnroot/pmd/trunk@222 51baf565-9d33-0410-a72c-fc3788e3496d

pmd/src/net/sourceforge/pmd/rules/UnnecessaryConversionTemporaryRule.java

@@ -31,6 +31,7 @@ public class UnnecessaryConversionTemporaryRule extends AbstractRule implements
         if (node.jjtGetNumChildren() == 0 || ((SimpleNode)node.jjtGetChild(0)).jjtGetNumChildren() == 0 || !(node.jjtGetChild(0).jjtGetChild(0) instanceof ASTAllocationExpression)) {
             return super.visit(node, data);
         }
+        // TODO... hmmm... is this inPrimaryExpressionContext gibberish necessary?
         inPrimaryExpressionContext = true;
         Object report = super.visit(node, data);
         inPrimaryExpressionContext = false;

pmd/src/net/sourceforge/pmd/rules/UnusedPrivateInstanceVariableRule.java

@@ -17,6 +17,9 @@ public class UnusedPrivateInstanceVariableRule extends AbstractRule implements R
     // TODO
     // this being an instance variable totally hoses up the recursion
     // need to attach it to the report or the stack or something
+    // I still need something to do forward references, though...
+    // and this "do the declarations first and the names second" works
+    // Actually, what I need is a Visitor that does a breadth first search
     // TODO
     private boolean doingIDTraversal;
     // TODO