Github user jmesnil commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1778#discussion_r161788189
--- Diff:
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQDestination.java
---
@@ -244,22 +249,30 @@ public static ActiveMQTemporaryTopic
createTemporaryTopic(String address) {
protected ActiveMQDestination(final String address,
final TYPE type,
final ActiveMQSession session) {
- this.simpleAddress = SimpleString.toSimpleString(address);
-
- this.thetype = type;
-
- this.session = session;
+ this(SimpleString.toSimpleString(address), type, session);
+ }
- this.temporary = TYPE.isTemporary(type);
+ protected ActiveMQDestination(final SimpleString address,
+ final TYPE type,
+ final ActiveMQSession session) {
+ this(address, null, type, session);
--- End diff --
should pass `address` instead of `null` for the name
---
