You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by ev...@apache.org on 2004/03/06 23:41:56 UTC
cvs commit: maven-plugins/jcoverage/src/main/org/apache/maven/jcoveragereport Coverage.java Package.java
evenisse 2004/03/06 14:41:56
Modified: jcoverage/src/main/org/apache/maven/jcoveragereport
Coverage.java Package.java
Log:
Fix possible NumberFormatException
Revision Changes Path
1.4 +22 -4 maven-plugins/jcoverage/src/main/org/apache/maven/jcoveragereport/Coverage.java
Index: Coverage.java
===================================================================
RCS file: /home/cvs/maven-plugins/jcoverage/src/main/org/apache/maven/jcoveragereport/Coverage.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- Coverage.java 2 Mar 2004 15:07:06 -0000 1.3
+++ Coverage.java 6 Mar 2004 22:41:56 -0000 1.4
@@ -129,8 +129,17 @@
{
for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
{
- Clazz theClass = (Clazz) iter.next();
- total += new Double(theClass.getBranchRate()).floatValue();
+ Clazz theClass = (Clazz) iter.next();
+ double rate = 0;
+ try
+ {
+ rate = new Double(theClass.getBranchRate()).floatValue();
+ }
+ catch(NumberFormatException e)
+ {
+ rate = 0;
+ }
+ total += rate;
}
}
@@ -143,7 +152,16 @@
for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
{
Clazz theClass = (Clazz) iter.next();
- total += new Double(theClass.getLineRate()).floatValue();
+ double rate = 0;
+ try
+ {
+ rate = new Double(theClass.getLineRate()).floatValue();
+ }
+ catch(NumberFormatException e)
+ {
+ rate = 0;
+ }
+ total += rate;
}
return total / getClasses().size();
}
1.5 +21 -3 maven-plugins/jcoverage/src/main/org/apache/maven/jcoveragereport/Package.java
Index: Package.java
===================================================================
RCS file: /home/cvs/maven-plugins/jcoverage/src/main/org/apache/maven/jcoveragereport/Package.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- Package.java 2 Mar 2004 15:07:06 -0000 1.4
+++ Package.java 6 Mar 2004 22:41:56 -0000 1.5
@@ -98,7 +98,16 @@
for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
{
Clazz theClass = (Clazz) iter.next();
- total += new Double(theClass.getBranchRate()).floatValue();
+ double rate = 0;
+ try
+ {
+ rate = new Double(theClass.getBranchRate()).floatValue();
+ }
+ catch(NumberFormatException e)
+ {
+ rate = 0;
+ }
+ total += rate;
}
total /= getClasses().size();
@@ -114,7 +123,16 @@
for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
{
Clazz theClass = (Clazz) iter.next();
- total += new Double(theClass.getLineRate()).floatValue();
+ double rate = 0;
+ try
+ {
+ rate = new Double(theClass.getLineRate()).floatValue();
+ }
+ catch(NumberFormatException e)
+ {
+ rate = 0;
+ }
+ total += rate;
}
return (total / getClasses().size());
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org