diff --git a/coming_results/all_instances_found.json b/coming_results/all_instances_found.json index 03e0fbff4..5e5406f58 100644 --- a/coming_results/all_instances_found.json +++ b/coming_results/all_instances_found.json @@ -1,157 +1,3 @@ { - "instances": [ - { - "revision": "cardumen", - "repairability": [ - { - "tool-name": "Cardumen", - "pattern-name": "Cardumen;expression_replacement", - "Unified_Diff_of-files:": "Starts Below...", - "CHANGE_old:": " minRatioPositions.add(i);", - "CHANGE_new:": " Precision.equals(entry, 0.0, maxUlps);", - "instance_detail": [ - { - "pattern_action": "UPD", - "pattern_entity": { - "entity_type": "Expression", - "entity_new value": "*", - "entity_role": "*", - "entity_parent": "null" - }, - "concrete_change": { - "operator": "UPD", - "src_type": "Invocation", - "dst_type": "Invocation", - "src": "minRatioPositions.add(i)", - "dst": "org.apache.commons.math3.util.Precision.equals(entry, 0.0, maxUlps)", - "src_parent_type": "Block", - "dst_parent_type": "Block", - "src_parent": "{\r\n minRatioPositions.add(i);\r\n}", - "dst_parent": "{\r\n org.apache.commons.math3.util.Precision.equals(entry, 0.0, maxUlps);\r\n}" - }, - "line": 104, - "file": "D:\\Daneshgah\\phd-kth\\projects\\commit summarization\\template-based\\coming\\two", - "stats": { - "number_of_src_entities": 16, - "number_of_dst_entities": 26, - "src_entity_types": "[java.lang.Integer, int, java.util.List\u003cjava.lang.Integer\u003e, java.util.List]", - "dst_entity_types": "[org.apache.commons.math3.util.Precision, double, org.apache.commons.math3.optimization.linear.SimplexSolver]" - } - } - ] - }, - { - "tool-name": "Cardumen", - "pattern-name": "Cardumen;expression_replacement", - "Unified_Diff_of-files:": "Starts Below...", - "CHANGE_old:": " minRatioPositions.add(i);", - "CHANGE_new:": " Precision.equals(entry, 0.0, maxUlps);", - "instance_detail": [ - { - "pattern_action": "UPD", - "pattern_entity": { - "entity_type": "Expression", - "entity_new value": "*", - "entity_role": "*", - "entity_parent": "null" - }, - "concrete_change": { - "operator": "UPD", - "src_type": "VariableRead", - "dst_type": "VariableRead", - "src": "minRatioPositions", - "dst": "entry", - "src_parent_type": "Invocation", - "dst_parent_type": "Invocation", - "src_parent": "minRatioPositions.add(i)", - "dst_parent": "org.apache.commons.math3.util.Precision.equals(entry, 0.0, maxUlps)" - }, - "line": 104, - "file": "D:\\Daneshgah\\phd-kth\\projects\\commit summarization\\template-based\\coming\\two", - "stats": { - "number_of_src_entities": 6, - "number_of_dst_entities": 3, - "src_entity_types": "[java.lang.Integer, java.util.List\u003cjava.lang.Integer\u003e]", - "dst_entity_types": "[double]" - } - } - ] - }, - { - "tool-name": "Cardumen", - "pattern-name": "Cardumen;expression_replacement", - "Unified_Diff_of-files:": "Starts Below...", - "CHANGE_old:": " minRatioPositions.add(i);", - "CHANGE_new:": " Precision.equals(entry, 0.0, maxUlps);", - "instance_detail": [ - { - "pattern_action": "UPD", - "pattern_entity": { - "entity_type": "Expression", - "entity_new value": "*", - "entity_role": "*", - "entity_parent": "null" - }, - "concrete_change": { - "operator": "UPD", - "src_type": "Invocation", - "dst_type": "Invocation", - "src": "minRatioPositions.add(i)", - "dst": "org.apache.commons.math3.util.Precision.equals(entry, 0.0, maxUlps)", - "src_parent_type": "Block", - "dst_parent_type": "Block", - "src_parent": "{\r\n minRatio \u003d ratio;\r\n minRatioPositions \u003d new java.util.ArrayList\u003cjava.lang.Integer\u003e();\r\n minRatioPositions.add(i);\r\n}", - "dst_parent": "{\r\n minRatio \u003d ratio;\r\n minRatioPositions \u003d new java.util.ArrayList\u003cjava.lang.Integer\u003e();\r\n org.apache.commons.math3.util.Precision.equals(entry, 0.0, maxUlps);\r\n}" - }, - "line": 108, - "file": "D:\\Daneshgah\\phd-kth\\projects\\commit summarization\\template-based\\coming\\two", - "stats": { - "number_of_src_entities": 16, - "number_of_dst_entities": 26, - "src_entity_types": "[java.lang.Integer, int, java.util.List\u003cjava.lang.Integer\u003e, java.util.List]", - "dst_entity_types": "[org.apache.commons.math3.util.Precision, double, org.apache.commons.math3.optimization.linear.SimplexSolver]" - } - } - ] - }, - { - "tool-name": "Cardumen", - "pattern-name": "Cardumen;expression_replacement", - "Unified_Diff_of-files:": "Starts Below...", - "CHANGE_old:": " minRatioPositions.add(i);", - "CHANGE_new:": " Precision.equals(entry, 0.0, maxUlps);", - "instance_detail": [ - { - "pattern_action": "UPD", - "pattern_entity": { - "entity_type": "Expression", - "entity_new value": "*", - "entity_role": "*", - "entity_parent": "null" - }, - "concrete_change": { - "operator": "UPD", - "src_type": "VariableRead", - "dst_type": "VariableRead", - "src": "minRatioPositions", - "dst": "entry", - "src_parent_type": "Invocation", - "dst_parent_type": "Invocation", - "src_parent": "minRatioPositions.add(i)", - "dst_parent": "org.apache.commons.math3.util.Precision.equals(entry, 0.0, maxUlps)" - }, - "line": 108, - "file": "D:\\Daneshgah\\phd-kth\\projects\\commit summarization\\template-based\\coming\\two", - "stats": { - "number_of_src_entities": 6, - "number_of_dst_entities": 3, - "src_entity_types": "[java.lang.Integer, java.util.List\u003cjava.lang.Integer\u003e]", - "dst_entity_types": "[double]" - } - } - ] - } - ] - } - ] + "instances": [] } \ No newline at end of file diff --git a/pom.xml b/pom.xml index 21baf394a..39a63ac4a 100644 --- a/pom.xml +++ b/pom.xml @@ -77,7 +77,7 @@ fr.inria.gforge.spoon.labs gumtree-spoon-ast-diff - 1.20 + 1.21 diff --git a/src/main/java/fr/inria/coming/repairability/repairtools/JGenProg.java b/src/main/java/fr/inria/coming/repairability/repairtools/JGenProg.java index e6865c026..f7a117ecf 100644 --- a/src/main/java/fr/inria/coming/repairability/repairtools/JGenProg.java +++ b/src/main/java/fr/inria/coming/repairability/repairtools/JGenProg.java @@ -50,7 +50,7 @@ public boolean filter(ChangePatternInstance instance, IRevision revision) { } else if (anyOperation instanceof UpdateOperation) { element = anyOperation.getDstNode(); // See why are using DstNode: https://github.com/SpoonLabs/coming/issues/72#issuecomment-508123273 } else if (anyOperation instanceof DeleteOperation) { - // ASSUMPTION ONLY A STATEMENT CAN BE DELETED + // ASSUMPTION: ONLY A STATEMENT CAN BE DELETED return anyOperation.getSrcNode().getRoleInParent() == CtRole.STATEMENT; } else if (anyOperation instanceof MoveOperation) { // based on move never occurs actually based on the analysis of our dataset but it may occur when in case of swaps(as described in the paper)