Cache the last used IProjectProperties to avoid a bunch of repeated checks and lookups.

Author: dkulp

net.sourceforge.pmd.eclipse.plugin/src/main/java/net/sourceforge/pmd/eclipse/runtime/cmd/ReviewCodeCmd.java

@@ -102,6 +102,8 @@ public class ReviewCodeCmd extends AbstractDefaultCommand {
     private int 						fileCount;
     private long 						pmdDuration;
     private String						onErrorIssue = null;
+    
+    private IProjectProperties          propertyCache = null;
 
     private static final long serialVersionUID = 1L;
 
@@ -353,11 +355,17 @@ private void processResources() throws CommandException {
             }
         }
     }
+    
+    private IProjectProperties getProjectProperties(IProject project) throws PropertiesException, CommandException {
+    	if (propertyCache == null || !propertyCache.getProject().getName().equals(project.getName())) {
+    		propertyCache = PMDPlugin.getDefault().loadProjectProperties(project);
+    	}
+		return propertyCache;
+    }
 
     private RuleSet rulesetFrom(IResource resource) throws PropertiesException, CommandException {
-    	
     	 IProject project = resource.getProject();
-         IProjectProperties properties = PMDPlugin.getDefault().loadProjectProperties(project);
+         IProjectProperties properties = getProjectProperties(project);
 
          return filteredRuleSet(properties);	//properties.getProjectRuleSet();
     }
@@ -369,7 +377,10 @@ private void processResource(IResource resource) throws CommandException {
         try {
 
             final IProject project = resource.getProject();
-            final IProjectProperties properties = PMDPlugin.getDefault().loadProjectProperties(project);
+            final IProjectProperties properties = getProjectProperties(project);
+            if (!properties.isPmdEnabled()) {
+            	return;
+            }
 
             final RuleSet ruleSet = rulesetFrom(resource);	//properties.getProjectRuleSet();
 
@@ -490,7 +501,7 @@ private RuleSet rulesetFromResourceDelta() throws PropertiesException, CommandEx
 
     	 IResource resource = resourceDelta.getResource();
          final IProject project = resource.getProject();
-         final IProjectProperties properties = PMDPlugin.getDefault().loadProjectProperties(project);
+         final IProjectProperties properties = getProjectProperties(project);
 
          return filteredRuleSet(properties);	//properties.getProjectRuleSet();
     }
@@ -502,7 +513,7 @@ private void processResourceDelta() throws CommandException {
         try {
             IResource resource = resourceDelta.getResource();
             final IProject project = resource.getProject();
-            final IProjectProperties properties = PMDPlugin.getDefault().loadProjectProperties(project);
+            final IProjectProperties properties = getProjectProperties(project);
 
             RuleSet ruleSet = rulesetFromResourceDelta();	//properties.getProjectRuleSet();
 
@@ -655,4 +666,4 @@ public boolean visit(IResourceDelta delta) {
         }
     }
 
-}
+}