From 44789e36eea9a549f0de482a856c3548bc0ef540 Mon Sep 17 00:00:00 2001 From: Par Winzell Date: Mon, 13 Apr 2009 17:28:37 +0000 Subject: [PATCH] Pointer equality between Doubles does not seem like a good idea. --- src/java/com/samskivert/depot/operator/Arithmetic.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/java/com/samskivert/depot/operator/Arithmetic.java b/src/java/com/samskivert/depot/operator/Arithmetic.java index 6db1982..adaba86 100644 --- a/src/java/com/samskivert/depot/operator/Arithmetic.java +++ b/src/java/com/samskivert/depot/operator/Arithmetic.java @@ -145,8 +145,8 @@ public abstract class Arithmetic extends MultiOperator @Override public Object evaluate (Object[] operands) { - for (int i = 1; i < operands.length; i ++) { - if (NUMERICAL.apply(operands[i]) == Double.valueOf(0)) { + for (int ii = 1; ii < operands.length; ii ++) { + if (Double.valueOf(0).equals(NUMERICAL.apply(operands[ii]))) { return new NoValue("Division by zero in: " + StringUtil.toString(operands)); } }