felix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pde...@apache.org
Subject svn commit: r930810 - /felix/trunk/dependencymanager/annotation/src/main/java/org/apache/felix/dm/annotation/plugin/mvn/AnnotationMojo.java
Date Mon, 05 Apr 2010 07:49:38 GMT
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);
+            }
         }
     }
 }



Mime
View raw message