search

barry-m / checker-framework

Automatically exported from code.google.com/p/checker-framework
Other
0 stars 0 forks source link

Commented-out code in JavacRunner causes NPE #190

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
JavacRunner provides more precise location information for reported problems. 
Thus, quick fixes only work with JavacRunner. I changed 
checkers.eclipse.actions.CheckerWorker.shouldUseJavacRunner() to return true. 
This switches the Eclipse plug-in from using CommandlineJavacRunner to 
JavacRunner. However, when I run a checker using the Eclipse plug-in, the 
plug-in throws the following exception:

eclipse.buildId=M20120914-1800
java.version=1.7.0_10
java.vendor=Oracle Corporation
BootLoader constants: OS=linux, ARCH=x86_64, WS=gtk, NL=en_US
Framework arguments:  -product org.eclipse.platform.ide

!ENTRY checker.framework.eclipse.plugin 4 0 2012-12-31 17:19:29.728
!MESSAGE Analysis exception
!STACK 0
java.lang.NullPointerException
    at org.eclipse.core.internal.runtime.Activator.getURLConverter(Activator.java:321)
    at org.eclipse.core.runtime.FileLocator.toFileURL(FileLocator.java:205)
    at checkers.eclipse.javac.JavacRunner.getLocation(JavacRunner.java:195)
    at checkers.eclipse.javac.JavacRunner.getOptions(JavacRunner.java:105)
    at checkers.eclipse.javac.JavacRunner.run(JavacRunner.java:74)
    at checkers.eclipse.actions.CheckerWorker.runChecker(CheckerWorker.java:125)
    at checkers.eclipse.actions.CheckerWorker.work(CheckerWorker.java:105)
    at checkers.eclipse.actions.CheckerWorker.run(CheckerWorker.java:83)
    at org.eclipse.core.internal.jobs.Worker.run(Worker.java:53)

Original issue reported on code.google.com by reprogra...@gmail.com on 31 Dec 2012 at 11:24

GoogleCodeExporter commented 9 years ago

Original comment by michael.ernst@gmail.com on 29 Jan 2013 at 6:28

GoogleCodeExporter commented 9 years ago 
Quick Fixes depend on this issue. Has there been any progress on this front?

Original comment by reprogra...@gmail.com on 12 Feb 2013 at 4:57

GoogleCodeExporter commented 9 years ago 
I will be working on Eclipse related changes soon and they will include these 
changes.  I will post here when the changes have been made. 

Thanks for your patience.

Original comment by Jonathan...@gmail.com on 12 Feb 2013 at 5:52

GoogleCodeExporter commented 9 years ago

Original comment by Jonathan...@gmail.com on 27 Feb 2013 at 8:57

GoogleCodeExporter commented 9 years ago

Original comment by michael.ernst@gmail.com on 1 Mar 2013 at 5:54

GoogleCodeExporter commented 9 years ago 
It's great to hear that issue #191 is fixed. Please let me know if I can help 
you in fixing this issue. Once this issue is fixed, I should be able to resume 
my work on the quick fixes.

Original comment by reprogra...@gmail.com on 15 Mar 2013 at 6:14