CMS supports bytes messages

[Nathan Mittler <na...@gmail.com>]

I know a few of people were waiting for feedback on this - so I'm sending to
both lists.  I've just submitted a couple of patches to the stomp C++ client
(CMS).  One fixes the way it was handling the broker url, and the other
fixes the handling of the bytes message.  It all looks good now - I tested
against the April 19th incubator snapshot of AMQ.  You'll need a recent
version of the broker as there have been a few patches made on the stomp
interface.

Regards,
Nate

Re: CMS supports bytes messages

[Hiram Chirino <hi...@hiramchirino.com>]

Hi Nathan,

I assume you've committed that patch to SVN by now.
I was just testing the latest cms against the latest activemq on OS X
and ran into an issue where the client was hanging in the 
StompTransport::addMessageListener call.

Root cause: the mutex being used did not seem to support recursive locking.

The following patch fixes the issue for me.  I'll commit it as soon as
SVN starts working again.
I'm not feeling very good about sprinkling all these if defs around
the code.  I'm sure we are going to have to update these constantly as
we support more platforms.  Would using autoconfig help us with these
issues?

Index: activemqcms/src/activemq/concurrent/Mutex.h
===================================================================
--- activemqcms/src/activemq/concurrent/Mutex.h (revision 396884)
+++ activemqcms/src/activemq/concurrent/Mutex.h (working copy)
@@ -47,9 +47,9 @@
            pthread_mutexattr_t attributes;
            pthread_mutexattr_init( &attributes );

-            #ifdef __USE_UNIX98
+       #if defined(__USE_UNIX98) || defined(__APPLE__)
                pthread_mutexattr_settype( &attributes,
PTHREAD_MUTEX_RECURSIVE );
-            #endif
+       #endif

            // Initialize the mutex.
            pthread_mutex_init( &mutex, &attributes );


On 4/23/06, Nathan Mittler <na...@gmail.com> wrote:
> I know a few of people were waiting for feedback on this - so I'm sending to
> both lists.  I've just submitted a couple of patches to the stomp C++ client
> (CMS).  One fixes the way it was handling the broker url, and the other
> fixes the handling of the bytes message.  It all looks good now - I tested
> against the April 19th incubator snapshot of AMQ.  You'll need a recent
> version of the broker as there have been a few patches made on the stomp
> interface.
>
> Regards,
> Nate
>
>


--
Regards,
Hiram