search

brunyuriy / quick-fix-scout

Automatically exported from code.google.com/p/quick-fix-scout
Other
1 stars 1 forks source link

Speculation Enchantment: Better Mapping of Problem Locations #19

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
At this moment, the hueristics for matching problem locations (between the 
shadow and original project) is quite messy (however it works). The below is a 
thrown (but handled internally) exception due to the complex check. 

Problem locations differ on args => arg1 = File3.D, arg2 = File1.D
WARNING: Cannot retrieve file for path = File2.D in 
(edu.washington.cs.util.eclipse.ResourceUtility#getFile)
Path must include project and resource name: /File2.D
    org.eclipse.core.runtime.Assert.isLegal(Assert.java:63)
    org.eclipse.core.internal.resources.Workspace.newResource(Workspace.java:2169)
    org.eclipse.core.internal.resources.Container.getFile(Container.java:208)
    edu.washington.cs.util.eclipse.ResourceUtility.getFile(ResourceUtility.java:359)
    edu.washington.cs.quickfix.speculation.model.SpeculationUtility.sameProblemLocationContent(SpeculationUtility.java:83)
    edu.washington.cs.quickfix.speculation.calc.SpeculationCalculator.doAnalysisPreparations(SpeculationCalculator.java:401)
    edu.washington.cs.quickfix.speculation.calc.SpeculationCalculator.doWork(SpeculationCalculator.java:351)
    edu.washington.cs.threading.MortalThread.run(MortalThread.java:88)

WARNING: Cannot retrieve file for path = File1.D in 
(edu.washington.cs.util.eclipse.ResourceUtility#getFile)
Path must include project and resource name: /File1.D
    org.eclipse.core.runtime.Assert.isLegal(Assert.java:63)
    org.eclipse.core.internal.resources.Workspace.newResource(Workspace.java:2169)
    org.eclipse.core.internal.resources.Container.getFile(Container.java:208)
    edu.washington.cs.util.eclipse.ResourceUtility.getFile(ResourceUtility.java:359)
    edu.washington.cs.quickfix.speculation.model.SpeculationUtility.sameProblemLocationContent(SpeculationUtility.java:84)
    edu.washington.cs.quickfix.speculation.calc.SpeculationCalculator.doAnalysisPreparations(SpeculationCalculator.java:401)
    edu.washington.cs.quickfix.speculation.calc.SpeculationCalculator.doWork(SpeculationCalculator.java:351)
    edu.washington.cs.threading.MortalThread.run(MortalThread.java:88)

Original issue reported on code.google.com by kivancmu...@gmail.com on 9 Dec 2011 at 12:38

GoogleCodeExporter commented 9 years ago 
Realized that the exception mentioned in the comment is totally normal and does 
not affect the matching power of the speculation. 

Removed unnecessary exception traces, added better comment that explains the 
behavior for matching. However, I still think that matching code needs a better 
implementation (I don't know how yet)

Original comment by kivancmu...@gmail.com on 9 Dec 2011 at 3:07