From 07f964813c71c7bf0dd5d4c1b7605242979f1fec Mon Sep 17 00:00:00 2001
From: TomyLobo <tomylobo@nurfuerspam.de>
Date: Sat, 25 Feb 2012 16:25:42 +0100
Subject: Reduced excessive exception handling in NumberConversions. Addresses
 BUKKIT-825

This also allows, for instance, to parse complex numbers with imaginary part=0, if the according toString method omits zero components.
This also saves some unboxing (Foo.valueOf returns a wrapper, while Foo.parseFoo returns a primitive)
---
 .../java/org/bukkit/util/NumberConversions.java    | 24 +++++++++++-----------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/src/main/java/org/bukkit/util/NumberConversions.java b/src/main/java/org/bukkit/util/NumberConversions.java
index b6cf1009..f70cc314 100644
--- a/src/main/java/org/bukkit/util/NumberConversions.java
+++ b/src/main/java/org/bukkit/util/NumberConversions.java
@@ -13,8 +13,8 @@ public final class NumberConversions {
             int result = 0;
 
             try {
-                result = Integer.valueOf((String) object);
-            } catch (Throwable ex) {}
+                result = Integer.parseInt(object.toString());
+            } catch (NumberFormatException ex) {}
 
             return result;
         }
@@ -27,8 +27,8 @@ public final class NumberConversions {
             float result = 0;
 
             try {
-                result = Float.valueOf((String) object);
-            } catch (Throwable ex) {}
+                result = Float.parseFloat(object.toString());
+            } catch (NumberFormatException ex) {}
 
             return result;
         }
@@ -41,8 +41,8 @@ public final class NumberConversions {
             double result = 0;
 
             try {
-                result = Double.valueOf((String) object);
-            } catch (Throwable ex) {}
+                result = Double.parseDouble(object.toString());
+            } catch (NumberFormatException ex) {}
 
             return result;
         }
@@ -55,8 +55,8 @@ public final class NumberConversions {
             long result = 0;
 
             try {
-                result = Long.valueOf((String) object);
-            } catch (Throwable ex) {}
+                result = Long.parseLong(object.toString());
+            } catch (NumberFormatException ex) {}
 
             return result;
         }
@@ -69,8 +69,8 @@ public final class NumberConversions {
             short result = 0;
 
             try {
-                result = Short.valueOf((String) object);
-            } catch (Throwable ex) {}
+                result = Short.parseShort(object.toString());
+            } catch (NumberFormatException ex) {}
 
             return result;
         }
@@ -83,8 +83,8 @@ public final class NumberConversions {
             byte result = 0;
 
             try {
-                result = Byte.valueOf((String) object);
-            } catch (Throwable ex) {}
+                result = Byte.parseByte(object.toString());
+            } catch (NumberFormatException ex) {}
 
             return result;
         }
-- 
cgit v1.2.3