Return-Path: Delivered-To: apmail-activemq-commits-archive@www.apache.org Received: (qmail 71840 invoked from network); 22 Oct 2008 12:45:51 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 22 Oct 2008 12:45:51 -0000 Received: (qmail 97997 invoked by uid 500); 22 Oct 2008 12:45:54 -0000 Delivered-To: apmail-activemq-commits-archive@activemq.apache.org Received: (qmail 97973 invoked by uid 500); 22 Oct 2008 12:45:54 -0000 Mailing-List: contact commits-help@activemq.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@activemq.apache.org Delivered-To: mailing list commits@activemq.apache.org Received: (qmail 97960 invoked by uid 99); 22 Oct 2008 12:45:54 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 22 Oct 2008 05:45:54 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 22 Oct 2008 12:44:52 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id BBA0D2388920; Wed, 22 Oct 2008 05:45:30 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r707059 - in /activemq/trunk/activemq-core/src: main/java/org/apache/activemq/command/ActiveMQStreamMessage.java test/java/org/apache/activemq/command/ActiveMQStreamMessageTest.java Date: Wed, 22 Oct 2008 12:45:30 -0000 To: commits@activemq.apache.org From: rajdavies@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20081022124530.BBA0D2388920@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: rajdavies Date: Wed Oct 22 05:45:30 2008 New Revision: 707059 URL: http://svn.apache.org/viewvc?rev=707059&view=rev Log: Fix for https://issues.apache.org/activemq/browse/AMQ-1965 Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQStreamMessage.java activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQStreamMessageTest.java Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQStreamMessage.java URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQStreamMessage.java?rev=707059&r1=707058&r2=707059&view=diff ============================================================================== --- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQStreamMessage.java (original) +++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQStreamMessage.java Wed Oct 22 05:45:30 2008 @@ -1087,7 +1087,9 @@ writeDouble(((Double)value).doubleValue()); } else if (value instanceof byte[]) { writeBytes((byte[])value); - } + }else if (value instanceof Long) { + writeLong(((Long)value).longValue()); + } } /** Modified: activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQStreamMessageTest.java URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQStreamMessageTest.java?rev=707059&r1=707058&r2=707059&view=diff ============================================================================== --- activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQStreamMessageTest.java (original) +++ activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQStreamMessageTest.java Wed Oct 22 05:45:30 2008 @@ -409,6 +409,11 @@ fail("Should have thrown exception"); } catch (MessageFormatException mfe) { } + msg = new ActiveMQStreamMessage(); + msg.writeObject(new Long("1")); + // reset so it's readable now + msg.reset(); + assertEquals(new Long("1"), msg.readObject()); } catch (JMSException jmsEx) { jmsEx.printStackTrace(); assertTrue(false); @@ -474,7 +479,7 @@ } } - public void testReadDouble() { + public void testReadDouble() { ActiveMQStreamMessage msg = new ActiveMQStreamMessage(); try { double test = 4.4d; @@ -535,6 +540,7 @@ jmsEx.printStackTrace(); assertTrue(false); } + } public void testReadString() {