fixed false positives of NPEfix on repogit4test data

SpoonLabs · Jul 18, 2019 · 6b11141 · 6b11141
1 parent 471d01d
commit 6b11141
Show file tree

Hide file tree

Showing 6 changed files with 36 additions and 13 deletions.
diff --git a/src/main/java/fr/inria/coming/core/engine/RevisionNavigationExperiment.java b/src/main/java/fr/inria/coming/core/engine/RevisionNavigationExperiment.java
@@ -7,20 +7,23 @@
 import fr.inria.coming.changeminer.entity.FinalResult;
 import fr.inria.coming.changeminer.entity.IRevision;
 import fr.inria.coming.core.engine.callback.IntermediateResultProcessorCallback;
+import fr.inria.coming.core.engine.filespair.FileDiff;
 import fr.inria.coming.core.entities.AnalysisResult;
 import fr.inria.coming.core.entities.RevisionDataset;
 import fr.inria.coming.core.entities.RevisionResult;
 import fr.inria.coming.core.entities.interfaces.IFilter;
 import fr.inria.coming.core.entities.interfaces.IOutput;
 import fr.inria.coming.core.entities.interfaces.RevisionOrder;
 import fr.inria.coming.main.ComingProperties;
+import org.apache.log4j.Logger;
 
 /**
  * 
  * @author Matias Martinez
  *
  */
 public abstract class RevisionNavigationExperiment<R extends IRevision> {
+	Logger log = Logger.getLogger(FileDiff.class.getName());
 
 	protected RevisionOrder<R> navigationStrategy = null;
 	protected List<Analyzer> analyzers = new ArrayList<>();
@@ -103,7 +106,8 @@ public FinalResult analyze() {
 
 			R oneRevision = iterator.next();
 
-			System.out.println("\n***********\nAnalyzing " + i + "/" + size);
+			log.info("\n***********\nAnalyzing " + i + "/" + size);
+//			System.out.println("\n***********\nAnalyzing " + i + "/" + size);
 			if (!accept(oneRevision)) {
 				continue;
 			}

diff --git a/src/main/java/fr/inria/coming/core/engine/files/FileDiff.java b/src/main/java/fr/inria/coming/core/engine/files/FileDiff.java
@@ -29,7 +29,8 @@ public FileDiff(File diffFolder) {
 	@Override
 	public List<IRevisionPair> getChildren() {
 		if (this.diffFolder == null) {
-			System.out.println("Diff folder == null");
+			log.info("Diff folder == null");
+//			System.out.println("Diff folder == null");
 			return null;
 		}
 		List<IRevisionPair> pairs = new ArrayList<>();
@@ -69,7 +70,8 @@ public List<IRevisionPair> getChildren() {
 				;
 			}
 		} catch (Exception e) {
-			System.err.println("Error analyzing " + diffFolder);
+			log.error("Error analyzing " + diffFolder);
+//			System.err.println("Error analyzing " + diffFolder);
 			e.printStackTrace();
 		}
 

diff --git a/src/main/java/fr/inria/coming/core/engine/filespair/FileDiff.java b/src/main/java/fr/inria/coming/core/engine/filespair/FileDiff.java
@@ -30,7 +30,8 @@ public FileDiff(File left, File right) {
     public List<IRevisionPair> getChildren() {
 
         if (left == null || right == null) {
-            System.out.println("files is null");
+            log.info("files is null");
+//            System.out.println("files is null");
             return null;
         }
 

diff --git a/src/main/java/fr/inria/coming/core/entities/DiffResult.java b/src/main/java/fr/inria/coming/core/entities/DiffResult.java
@@ -1,5 +1,8 @@
 package fr.inria.coming.core.entities;
 
+import fr.inria.coming.changeminer.analyzer.commitAnalyzer.FineGrainDifftAnalyzer;
+import org.apache.log4j.Logger;
+
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
@@ -13,6 +16,8 @@
  */
 public class DiffResult<T, R> extends AnalysisResult<T> {
 
+	static Logger log = Logger.getLogger(FineGrainDifftAnalyzer.class.getName());
+
 	/**
 	 * Filename
 	 */
@@ -50,7 +55,9 @@ public String toString() {
 			try {
 				r += "\n" + idiff.toString();
 			} catch (Exception e) {
-				System.err.println("Error when printing diff result: " + e.getMessage());
+
+//				System.err.println("Error when printing diff result: " + e.getMessage());
+				log.error("Error when printing diff result: " + e.getMessage());
 				e.printStackTrace();
 			}
 

diff --git a/src/main/java/fr/inria/coming/main/ComingMain.java b/src/main/java/fr/inria/coming/main/ComingMain.java
@@ -55,7 +55,7 @@
  */
 public class ComingMain {
 
-	Logger log = Logger.getLogger(FineGrainDifftAnalyzer.class.getName());
+	static Logger logm = Logger.getLogger(FineGrainDifftAnalyzer.class.getName());
 
 	static Options options = new Options();
 
@@ -123,8 +123,10 @@ public static void main(String[] args) {
 		try {
 			cmain.run(args);
 		} catch (Exception e) {
-			System.err.println("Error initializing Coming with args" + Arrays.toString(args));
-			System.err.println(e);
+			logm.error("Error initializing Coming with args\" + Arrays.toString(args)");
+//			System.err.println("Error initializing Coming with args" + Arrays.toString(args));
+			logm.error(e);
+//			System.err.println(e);
 			e.printStackTrace();
 		}
 	}
@@ -157,7 +159,8 @@ public boolean createEngine(String[] args) throws ParseException {
 		try {
 			cmd = parser.parse(options, args);
 		} catch (UnrecognizedOptionException e) {
-			System.out.println("Error: " + e.getMessage());
+			logm.error("Error: " + e.getMessage());
+//			System.out.println("Error: " + e.getMessage());
 			help();
 			return false;
 		}
@@ -413,7 +416,7 @@ private List<ChangePatternSpecification> loadPattern() {
 					ChangePatternSpecification patternParsed = patternParser.parse(fl);
 					patternsFound.add(patternParsed);
 				} else {
-					log.error("The pattern file given as input does not exist " + fl.getAbsolutePath());
+					logm.error("The pattern file given as input does not exist " + fl.getAbsolutePath());
 				}
 			}
 		} else {
@@ -470,7 +473,8 @@ private static void help() {
 
 		HelpFormatter formater = new HelpFormatter();
 		formater.printHelp("Main", options);
-		System.out.println("More options and default values at 'configuration.properties' file");
+		logm.info("More options and default values at 'configuration.properties' file");
+//		System.out.println("More options and default values at 'configuration.properties' file");
 
 		System.exit(0);
 

diff --git a/src/main/java/fr/inria/coming/repairability/JSONRepairabilityOutput.java b/src/main/java/fr/inria/coming/repairability/JSONRepairabilityOutput.java
@@ -2,6 +2,7 @@
 
 import com.google.gson.JsonArray;
 import com.google.gson.JsonObject;
+import fr.inria.coming.changeminer.analyzer.commitAnalyzer.FineGrainDifftAnalyzer;
 import fr.inria.coming.changeminer.analyzer.instancedetector.ChangePatternInstance;
 import fr.inria.coming.changeminer.analyzer.instancedetector.PatternInstanceAnalyzer;
 import fr.inria.coming.changeminer.analyzer.instancedetector.PatternInstancesFromDiff;
@@ -10,11 +11,13 @@
 import fr.inria.coming.core.entities.RevisionResult;
 import fr.inria.coming.core.entities.output.JSonPatternInstanceOutput;
 import gumtree.spoon.diff.operations.Operation;
+import org.apache.log4j.Logger;
 
 import java.io.File;
 
 public class JSONRepairabilityOutput extends JSonPatternInstanceOutput {
 
+    Logger log = Logger.getLogger(FineGrainDifftAnalyzer.class.getName());
 
     public void getInstancesOfRevision(RevisionResult revisionResult, JsonArray instances) {
 
@@ -33,8 +36,10 @@ public void getInstancesOfRevision(RevisionResult revisionResult, JsonArray inst
 
                 instance.addProperty("revision", revisionIdentifier.toString());
 
-                System.out.println("\n--------\ncommit with instance:\n " + revisionIdentifier);
-                System.out.println(pi.getInstances());
+                log.info("\n--------\ncommit with instance:\n " + revisionIdentifier);
+//                System.out.println("\n--------\ncommit with instance:\n " + revisionIdentifier);
+                log.info(pi.getInstances());
+//                System.out.println(pi.getInstances());
 
                 JsonArray repair_tools = new JsonArray();
                 for (ChangePatternInstance instancePattern : pi.getInstances()) {