Re: MAP_TYPE support in ActiveMQ CPP

2008-03-10 Thread stevenbohrer
Hi Tim, I tried the suggested fix, but that seems to address the outgoing MapMessage. Regardless, it didn't resolve my issue. As far as I can tell through the debugger, I'm receiving the map message fine and unmarshaling the data into the PrimitiveMap properties private variable OK. But since

Re: MAP_TYPE support in ActiveMQ CPP

2008-03-10 Thread Timothy Bish
If you create patches you can attach them to this issue: https://issues.apache.org/activemq/browse/AMQCPP-172 You can compare the current version of ActiveMQMapMessage to the previous one in SVN to find the changes. Version 2.2 is not yet released. To save you some time, here is the basic fix c

Re: MAP_TYPE support in ActiveMQ CPP

2008-03-10 Thread stevenbohrer
Hi Tim, I've implemented the Map and List types now, and will work on submitting them with the proper tests, etc. However, I have another issue, I'm using a Topic with MapMessages, and they don't seem to be received properly. I found https://issues.apache.org/activemq/browse/AMQCPP-166 which se

Re: MAP_TYPE support in ActiveMQ CPP

2008-03-06 Thread Timothy Bish
A list is just what it sounds like. If you look at the code in the Java src you can see that they are just written as a series of primitives, with a leading size. Should be pretty simple to take this logic and insert it into the PrimitiveMapMarshaler. public static void marshalPrimitiveList

Re: MAP_TYPE support in ActiveMQ CPP

2008-03-06 Thread stevenbohrer
Tim, As suggested, I'll continue this correspondence in this thread. I decided to just modify the PrimitiveMap.* and MarshalPrimitiveMap.* files in a manner consistent with what had already been done. The java example I saw didn't mesh well with the cpp files. This is now completed and seems

Re: MAP_TYPE support in ActiveMQ CPP

2008-03-06 Thread Timothy Bish
We are always willing to accept code contributions. :) Currently there is no one working on that, you could create a Jira issue to document that someone is interested in such support and it may get added in the future. Looking at the map and list encoding it should be pretty easy to implement, I