From c4468c342efcd45bdc6cc08ea9c06ea6edd75f69 Mon Sep 17 00:00:00 2001 From: Andrew John Hughes Date: Mon, 23 Jun 2008 20:59:32 +0000 Subject: 2008-06-23 Andrew John Hughes * gnu/xml/xpath/CountFunction.java, * gnu/xml/xpath/EqualityExpr.java, * gnu/xml/xpath/Expr.java, * gnu/xml/xpath/IdFunction.java, * gnu/xml/xpath/LocalNameFunction.java, * gnu/xml/xpath/NameFunction.java, * gnu/xml/xpath/NamespaceUriFunction.java, * gnu/xml/xpath/ParenthesizedExpr.java, * gnu/xml/xpath/Steps.java, * gnu/xml/xpath/SumFunction.java, * gnu/xml/xpath/UnionExpr.java, * gnu/xml/xpath/XPathParser.java, * gnu/xml/xpath/XPathParser.y, * java/lang/Enum.java, * java/lang/reflect/Constructor.java, * java/lang/reflect/Field.java, * java/lang/reflect/Method.java: Reduce scope of unchecked warning suppression, and remove unneeded uses. --- gnu/xml/xpath/SumFunction.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'gnu/xml/xpath/SumFunction.java') diff --git a/gnu/xml/xpath/SumFunction.java b/gnu/xml/xpath/SumFunction.java index ce2989b4b..73db2a91d 100644 --- a/gnu/xml/xpath/SumFunction.java +++ b/gnu/xml/xpath/SumFunction.java @@ -66,14 +66,18 @@ final class SumFunction this.arg = arg; } - @Override @SuppressWarnings("unchecked") + @Override public Object evaluate(Node context, int pos, int len) { Object val = arg.evaluate(context, pos, len); double sum = 0.0d; if (val instanceof Collection) { - for (Node node : ((Collection) val)) + /* Suppression is safe, as we know context produces + Collection */ + @SuppressWarnings("unchecked") + Collection nodes = (Collection) val; + for (Node node : nodes) { String s = stringValue(node); sum += _number(context, s); -- cgit v1.2.1