QPIDJMS-75: remove unused constructor arg
Project: http://git-wip-us.apache.org/repos/asf/qpid-jms/repo Commit: http://git-wip-us.apache.org/repos/asf/qpid-jms/commit/033f0a28 Tree: http://git-wip-us.apache.org/repos/asf/qpid-jms/tree/033f0a28 Diff: http://git-wip-us.apache.org/repos/asf/qpid-jms/diff/033f0a28 Branch: refs/heads/master Commit: 033f0a2869cbd962e2a593d149eac37032c57553 Parents: 78b8ea7 Author: Robert Gemmell <[email protected]> Authored: Mon Jun 22 15:23:57 2015 +0100 Committer: Robert Gemmell <[email protected]> Committed: Mon Jun 22 17:28:15 2015 +0100 ---------------------------------------------------------------------- .../testpeer/AbstractFrameFieldAndPayloadMatchingHandler.java | 4 +--- .../jms/test/testpeer/FrameWithNoPayloadMatchingHandler.java | 5 ++--- .../qpid/jms/test/testpeer/FrameWithPayloadMatchingHandler.java | 5 ++--- .../apache/qpid/jms/test/testpeer/matchers/AttachMatcher.java | 3 +-- .../apache/qpid/jms/test/testpeer/matchers/BeginMatcher.java | 3 +-- .../apache/qpid/jms/test/testpeer/matchers/CloseMatcher.java | 3 +-- .../apache/qpid/jms/test/testpeer/matchers/DetachMatcher.java | 3 +-- .../qpid/jms/test/testpeer/matchers/DispositionMatcher.java | 3 +-- .../org/apache/qpid/jms/test/testpeer/matchers/EndMatcher.java | 3 +-- .../org/apache/qpid/jms/test/testpeer/matchers/FlowMatcher.java | 3 +-- .../org/apache/qpid/jms/test/testpeer/matchers/OpenMatcher.java | 3 +-- .../qpid/jms/test/testpeer/matchers/SaslChallengeMatcher.java | 3 +-- .../apache/qpid/jms/test/testpeer/matchers/SaslInitMatcher.java | 3 +-- .../qpid/jms/test/testpeer/matchers/SaslMechanismsMatcher.java | 3 +-- .../qpid/jms/test/testpeer/matchers/SaslOutcomeMatcher.java | 3 +-- .../qpid/jms/test/testpeer/matchers/SaslResponseMatcher.java | 3 +-- .../apache/qpid/jms/test/testpeer/matchers/TransferMatcher.java | 3 +-- .../qpid/jms/test/testpeer/matchers/generate-matchers.xsl | 3 +-- 18 files changed, 20 insertions(+), 39 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/AbstractFrameFieldAndPayloadMatchingHandler.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/AbstractFrameFieldAndPayloadMatchingHandler.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/AbstractFrameFieldAndPayloadMatchingHandler.java index 2b512ed..ec1c2c9 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/AbstractFrameFieldAndPayloadMatchingHandler.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/AbstractFrameFieldAndPayloadMatchingHandler.java @@ -44,13 +44,11 @@ public abstract class AbstractFrameFieldAndPayloadMatchingHandler extends Abstra protected AbstractFrameFieldAndPayloadMatchingHandler(FrameType frameType, int channel, UnsignedLong numericDescriptor, - Symbol symbolicDescriptor, - AmqpPeerRunnable onCompletion) + Symbol symbolicDescriptor) { super(numericDescriptor, symbolicDescriptor); _frameType = frameType; _expectedChannel = channel; - _onCompletion = onCompletion; } protected abstract void verifyPayload(Binary payload) throws AssertionError; http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithNoPayloadMatchingHandler.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithNoPayloadMatchingHandler.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithNoPayloadMatchingHandler.java index 63cc2dd..1d2feb1 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithNoPayloadMatchingHandler.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithNoPayloadMatchingHandler.java @@ -28,10 +28,9 @@ public class FrameWithNoPayloadMatchingHandler extends AbstractFrameFieldAndPayl protected FrameWithNoPayloadMatchingHandler(FrameType frameType, int channel, UnsignedLong numericDescriptor, - Symbol symbolicDescriptor, - AmqpPeerRunnable onSuccess) + Symbol symbolicDescriptor) { - super(frameType, channel, numericDescriptor, symbolicDescriptor, onSuccess); + super(frameType, channel, numericDescriptor, symbolicDescriptor); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithPayloadMatchingHandler.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithPayloadMatchingHandler.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithPayloadMatchingHandler.java index d9ac6af..fcfbc33 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithPayloadMatchingHandler.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/FrameWithPayloadMatchingHandler.java @@ -33,10 +33,9 @@ public class FrameWithPayloadMatchingHandler extends AbstractFrameFieldAndPayloa protected FrameWithPayloadMatchingHandler(FrameType frameType, int channel, UnsignedLong numericDescriptor, - Symbol symbolicDescriptor, - AmqpPeerRunnable onCompletion) + Symbol symbolicDescriptor) { - super(frameType, channel, numericDescriptor, symbolicDescriptor, onCompletion); + super(frameType, channel, numericDescriptor, symbolicDescriptor); } public void setPayloadMatcher(Matcher<Binary> payloadMatcher) http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/AttachMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/AttachMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/AttachMatcher.java index e4695a8..0809a44 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/AttachMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/AttachMatcher.java @@ -56,8 +56,7 @@ public class AttachMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000012L), - Symbol.valueOf("amqp:attach:list"), - null); + Symbol.valueOf("amqp:attach:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/BeginMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/BeginMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/BeginMatcher.java index fffbbb8..b660f63 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/BeginMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/BeginMatcher.java @@ -50,8 +50,7 @@ public class BeginMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000011L), - Symbol.valueOf("amqp:begin:list"), - null); + Symbol.valueOf("amqp:begin:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/CloseMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/CloseMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/CloseMatcher.java index d162ba1..0442232 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/CloseMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/CloseMatcher.java @@ -43,8 +43,7 @@ public class CloseMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000018L), - Symbol.valueOf("amqp:close:list"), - null); + Symbol.valueOf("amqp:close:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DetachMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DetachMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DetachMatcher.java index 4d49d57..0254c32 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DetachMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DetachMatcher.java @@ -45,8 +45,7 @@ public class DetachMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000016L), - Symbol.valueOf("amqp:detach:list"), - null); + Symbol.valueOf("amqp:detach:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DispositionMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DispositionMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DispositionMatcher.java index d18ea96..a95a027 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DispositionMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/DispositionMatcher.java @@ -48,8 +48,7 @@ public class DispositionMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000015L), - Symbol.valueOf("amqp:disposition:list"), - null); + Symbol.valueOf("amqp:disposition:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/EndMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/EndMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/EndMatcher.java index 34cb6d7..076fd70 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/EndMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/EndMatcher.java @@ -43,8 +43,7 @@ public class EndMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000017L), - Symbol.valueOf("amqp:end:list"), - null); + Symbol.valueOf("amqp:end:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/FlowMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/FlowMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/FlowMatcher.java index 145d447..c7667a8 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/FlowMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/FlowMatcher.java @@ -53,8 +53,7 @@ public class FlowMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000013L), - Symbol.valueOf("amqp:flow:list"), - null); + Symbol.valueOf("amqp:flow:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/OpenMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/OpenMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/OpenMatcher.java index 4d1693d..aaed6a0 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/OpenMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/OpenMatcher.java @@ -52,8 +52,7 @@ public class OpenMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000010L), - Symbol.valueOf("amqp:open:list"), - null); + Symbol.valueOf("amqp:open:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslChallengeMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslChallengeMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslChallengeMatcher.java index ae5280d..99bf0c4 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslChallengeMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslChallengeMatcher.java @@ -43,8 +43,7 @@ public class SaslChallengeMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.SASL, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000042L), - Symbol.valueOf("amqp:sasl-challenge:list"), - null); + Symbol.valueOf("amqp:sasl-challenge:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslInitMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslInitMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslInitMatcher.java index 373d579..c777a66 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslInitMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslInitMatcher.java @@ -45,8 +45,7 @@ public class SaslInitMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.SASL, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000041L), - Symbol.valueOf("amqp:sasl-init:list"), - null); + Symbol.valueOf("amqp:sasl-init:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslMechanismsMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslMechanismsMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslMechanismsMatcher.java index 7dbaabd..482c19f 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslMechanismsMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslMechanismsMatcher.java @@ -43,8 +43,7 @@ public class SaslMechanismsMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.SASL, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000040L), - Symbol.valueOf("amqp:sasl-mechanisms:list"), - null); + Symbol.valueOf("amqp:sasl-mechanisms:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslOutcomeMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslOutcomeMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslOutcomeMatcher.java index da4280c..4fb0ee8 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslOutcomeMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslOutcomeMatcher.java @@ -44,8 +44,7 @@ public class SaslOutcomeMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.SASL, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000044L), - Symbol.valueOf("amqp:sasl-outcome:list"), - null); + Symbol.valueOf("amqp:sasl-outcome:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslResponseMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslResponseMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslResponseMatcher.java index 050a9e5..28184a8 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslResponseMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/SaslResponseMatcher.java @@ -43,8 +43,7 @@ public class SaslResponseMatcher extends FrameWithNoPayloadMatchingHandler super(FrameType.SASL, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000043L), - Symbol.valueOf("amqp:sasl-response:list"), - null); + Symbol.valueOf("amqp:sasl-response:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/TransferMatcher.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/TransferMatcher.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/TransferMatcher.java index 9fb02bd..17e3eb3 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/TransferMatcher.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/TransferMatcher.java @@ -53,8 +53,7 @@ public class TransferMatcher extends FrameWithPayloadMatchingHandler super(FrameType.AMQP, ANY_CHANNEL, UnsignedLong.valueOf(0x0000000000000014L), - Symbol.valueOf("amqp:transfer:list"), - null); + Symbol.valueOf("amqp:transfer:list")); } @Override http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/033f0a28/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/generate-matchers.xsl ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/generate-matchers.xsl b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/generate-matchers.xsl index 18a1b97..ceb6f70 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/generate-matchers.xsl +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/test/testpeer/matchers/generate-matchers.xsl @@ -92,8 +92,7 @@ public class <xsl:value-of select="$classname"/> extends <xsl:value-of select="$ super(FrameType.<xsl:choose><xsl:when test="@provides='sasl-frame'">SASL</xsl:when><xsl:otherwise>AMQP</xsl:otherwise></xsl:choose>, ANY_CHANNEL, UnsignedLong.valueOf(<xsl:value-of select="concat(substring(descendant::node()[name()='descriptor']/@code,1,10),substring(descendant::node()[name()='descriptor']/@code,14))"/>L), - Symbol.valueOf("<xsl:value-of select="descendant::node()[name()='descriptor']/@name"/>"), - null); + Symbol.valueOf("<xsl:value-of select="descendant::node()[name()='descriptor']/@name"/>")); } @Override --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
