Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

NO-JIRA Remove redundant instanceof check in CoreAmqpConverter 

message is of type ServerJMSMessage, thus the test `if (message instanceof ServerJMSMessage)` always yields true.
This commit is contained in:
Sebastian Thomschke 2020-01-31 10:05:51 +01:00 committed by Justin Bertram
parent 34e58a9f40
commit 26b5ad439c
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/converter/CoreAmqpConverter.java
View File

@ -493,7 +493,7 @@ public class CoreAmqpConverter {
if (!message.propertyExists(JMS_AMQP_CONTENT_TYPE)) { if (!message.propertyExists(JMS_AMQP_CONTENT_TYPE)) {
message.setStringProperty(JMS_AMQP_CONTENT_TYPE, SERIALIZED_JAVA_OBJECT_CONTENT_TYPE.toString()); message.setStringProperty(JMS_AMQP_CONTENT_TYPE, SERIALIZED_JAVA_OBJECT_CONTENT_TYPE.toString());
} }
} else if (message instanceof ServerJMSMessage) { } else {
maMap.put(AMQPMessageSupport.JMS_MSG_TYPE, AMQPMessageSupport.JMS_MESSAGE); maMap.put(AMQPMessageSupport.JMS_MSG_TYPE, AMQPMessageSupport.JMS_MESSAGE);
// If this is not an AMQP message that was converted then the original encoding // If this is not an AMQP message that was converted then the original encoding
// will be unknown so we check for special cases of messages with special data // will be unknown so we check for special cases of messages with special data