You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pd...@apache.org on 2010/04/05 09:49:38 UTC
svn commit: r930810 -
/felix/trunk/dependencymanager/annotation/src/main/java/org/apache/felix/dm/annotation/plugin/mvn/AnnotationMojo.java
Author: pderop
Date: Mon Apr 5 07:49:38 2010
New Revision: 930810
URL: http://svn.apache.org/viewvc?rev=930810&view=rev
Log:
fixed findbugs warning: File.exists() and File.delete() return value was not checked
Modified:
felix/trunk/dependencymanager/annotation/src/main/java/org/apache/felix/dm/annotation/plugin/mvn/AnnotationMojo.java
Modified: felix/trunk/dependencymanager/annotation/src/main/java/org/apache/felix/dm/annotation/plugin/mvn/AnnotationMojo.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/annotation/src/main/java/org/apache/felix/dm/annotation/plugin/mvn/AnnotationMojo.java?rev=930810&r1=930809&r2=930810&view=diff
==============================================================================
--- felix/trunk/dependencymanager/annotation/src/main/java/org/apache/felix/dm/annotation/plugin/mvn/AnnotationMojo.java (original)
+++ felix/trunk/dependencymanager/annotation/src/main/java/org/apache/felix/dm/annotation/plugin/mvn/AnnotationMojo.java Mon Apr 5 07:49:38 2010
@@ -186,7 +186,9 @@ public class AnnotationMojo extends Abst
finally
{
jar.close();
- tmp.delete();
+ if (tmp.exists() && ! tmp.delete()) {
+ throw new MojoExecutionException("Could not remove " + tmp);
+ }
}
}
}