cshannon commented on code in PR #979:
URL: https://github.com/apache/activemq/pull/979#discussion_r1108750673
##########
activemq-client/src/main/java/org/apache/activemq/command/ActiveMQMapMessage.java:
##########
@@ -827,4 +839,26 @@ public Map<String, Object> getContentMap() throws
JMSException {
initializeReading();
return map;
}
+
+ @Override
+ @SuppressWarnings("unchecked")
+ public boolean isBodyAssignableTo(Class c) throws JMSException {
+ final Map<String, Object> map = getContentMap();
+ if (map == null || map.isEmpty()) {
Review Comment:
Good catch, it likely should because the map message impl treats an empty
map as basically null (no body) so if it's empty we should return null I think
and not an empty map.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
