2 files changed, 21 insertions, 64 deletions

diff --git a/java/common/src/main/java/org/apache/qpid/framing/JMSPropertyFieldTable.java b/java/common/src/main/java/org/apache/qpid/framing/JMSPropertyFieldTable.java
index cc56e7c4de..d2b64ec3c5 100644
--- a/java/common/src/main/java/org/apache/qpid/framing/JMSPropertyFieldTable.java
+++ b/java/common/src/main/java/org/apache/qpid/framing/JMSPropertyFieldTable.java
@@ -205,8 +205,6 @@ public class JMSPropertyFieldTable
 
     public byte getByte(String string) throws JMSException
     {
-        try
-        {
             Byte b = _fieldtable.getByte(string);
             if (b == null)
             {
@@ -230,17 +228,10 @@ public class JMSPropertyFieldTable
             }
 
             return b;
-        }
-        catch (NumberFormatException nfe)
-        {
-            throw new MessageFormatException(nfe.getMessage());
-        }
     }
 
     public short getShort(String string) throws JMSException
     {
-        try
-        {
             Short s = _fieldtable.getShort(string);
 
             if (s == null)
@@ -249,17 +240,10 @@ public class JMSPropertyFieldTable
             }
 
             return s;
-        }
-        catch (NumberFormatException nfe)
-        {
-            throw new MessageFormatException(nfe.getMessage());
-        }
     }
 
     public int getInteger(String string) throws JMSException
     {
-        try
-        {
             Integer i = _fieldtable.getInteger(string);
 
             if (i == null)
@@ -268,17 +252,10 @@ public class JMSPropertyFieldTable
             }
 
             return i;
-        }
-        catch (NumberFormatException nfe)
-        {
-            throw new MessageFormatException(nfe.getMessage());
-        }
     }
 
     public long getLong(String string) throws JMSException
     {
-        try
-        {
             Long l = _fieldtable.getLong(string);
 
             if (l == null)
@@ -287,18 +264,10 @@ public class JMSPropertyFieldTable
             }
 
             return l;
-        }
-        catch (NumberFormatException nfe)
-        {
-            throw new MessageFormatException(nfe.getMessage());
-        }
-
     }
 
     public float getFloat(String string) throws JMSException
     {
-        try
-        {
             Float f = _fieldtable.getFloat(string);
 
             if (f == null)
@@ -324,17 +293,10 @@ public class JMSPropertyFieldTable
             }
 
             return f;
-        }
-        catch (NumberFormatException nfe)
-        {
-            throw new MessageFormatException(nfe.getMessage());
-        }
     }
 
     public double getDouble(String string) throws JMSException
     {
-        try
-        {
             Double d = _fieldtable.getDouble(string);
 
             if (d == null)
@@ -342,12 +304,7 @@ public class JMSPropertyFieldTable
                 d = Double.valueOf(getFloat(string));
             }
 
-            return d;
-        }
-        catch (NumberFormatException nfe)
-        {
-            throw new MessageFormatException(nfe.getMessage());
-        }
+            return d;        
     }
 
     public String getString(String string) throws JMSException
diff --git a/java/common/src/test/java/org/apache/qpid/framing/JMSPropertyFieldTableTest.java b/java/common/src/test/java/org/apache/qpid/framing/JMSPropertyFieldTableTest.java
index 98202ddd1a..865735499b 100644
--- a/java/common/src/test/java/org/apache/qpid/framing/JMSPropertyFieldTableTest.java
+++ b/java/common/src/test/java/org/apache/qpid/framing/JMSPropertyFieldTableTest.java
@@ -254,9 +254,9 @@ public class JMSPropertyFieldTableTest extends TestCase
         try
         {
             table1.getByte("Rubbish");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException mfs)
+        catch (NumberFormatException mfs)
         {
             //normal Execution
         }
@@ -334,9 +334,9 @@ public class JMSPropertyFieldTableTest extends TestCase
         try
         {
             table1.getShort("Rubbish");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException mfe)
+        catch (NumberFormatException mfe)
         {
             //normal path
         }
@@ -597,9 +597,9 @@ public class JMSPropertyFieldTableTest extends TestCase
         try
         {
             table1.getInteger("Rubbish");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException mfe)
+        catch (NumberFormatException mfe)
         {
             //normal path
         }
@@ -690,9 +690,9 @@ public class JMSPropertyFieldTableTest extends TestCase
         try
         {
             table1.getLong("Rubbish");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException mfs)
+        catch (NumberFormatException mfs)
         {
             //normal Execution
         }
@@ -731,54 +731,54 @@ public class JMSPropertyFieldTableTest extends TestCase
         try
         {
             table1.getByte("value");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException nfs)
+        catch (NumberFormatException nfs)
         {
             //normal Execution
         }
         try
         {
             table1.getShort("value");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException nfs)
+        catch (NumberFormatException nfs)
         {
             //normal Execution
         }
         try
         {
             table1.getDouble("value");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException nfs)
+        catch (NumberFormatException nfs)
         {
             //normal Execution
         }
         try
         {
             table1.getFloat("value");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException nfs)
+        catch (NumberFormatException nfs)
         {
             //normal Execution
         }
         try
         {
             table1.getInteger("value");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException nfs)
+        catch (NumberFormatException nfs)
         {
             //normal Execution
         }
         try
         {
             table1.getLong("value");
-            fail("Should throw MessageFormatException");
+            fail("Should throw NumberFormatException");
         }
-        catch (MessageFormatException nfs)
+        catch (NumberFormatException nfs)
         {
             //normal Execution
         }