Remove unnecessary semicolons

tests/activemq5-unit-tests/src/main/java/org/apache/activemq/ActiveMQConnectionFactory.java

@@ -209,7 +209,6 @@ public class ActiveMQConnectionFactory extends JNDIBaseStorable implements Conne
             params.clear();
 
             this.vmBrokerUri = URISupport.createRemainingURI(uri, params);
-            ;
          }
       }
       catch (URISyntaxException e) {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/JMSConsumerTest.java

@@ -168,7 +168,6 @@ public class JMSConsumerTest extends JmsTestSupport {
             }
          }
       }
-      ;
 
       final ExecutorService executor = Executors.newCachedThreadPool();
       consumer.setMessageListener(new MessageListener() {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/JmsQueueBrowserTest.java

@@ -121,7 +121,6 @@ public class JmsQueueBrowserTest extends JmsTestSupport {
       for (int i = 0; i < 10; i++) {
          outbound[i] = session.createTextMessage(i + " Message");
       }
-      ;
 
       // lets consume any outstanding messages from previous test runs
       while (consumer.receive(1000) != null) {
@@ -178,7 +177,6 @@ public class JmsQueueBrowserTest extends JmsTestSupport {
       for (int i = 0; i < 10; i++) {
          outbound[i] = session.createTextMessage(i + " Message");
       }
-      ;
 
       // lets consume any outstanding messages from previous test runs
       while (consumer.receive(1000) != null) {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/JmsSendReceiveWithMessageExpirationTest.java

@@ -154,7 +154,6 @@ public class JmsSendReceiveWithMessageExpirationTest extends TestSupport {
          }
          received.acknowledge();
       }
-      ;
 
       assertEquals("got all (normal plus one with ttl) messages", messageCount + 1, messages.size());
 
@@ -162,7 +161,6 @@ public class JmsSendReceiveWithMessageExpirationTest extends TestSupport {
       while ((received = dlqConsumer.receive(1000)) != null) {
          dlqMessages.add(received);
       }
-      ;
 
       assertEquals("got dlq messages", data.length - 1, dlqMessages.size());
 

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/SpringTestSupport.java

@@ -39,8 +39,6 @@ public abstract class SpringTestSupport extends TestCase {
 
    protected abstract AbstractApplicationContext createApplicationContext();
 
-   ;
-
    @Override
    protected void tearDown() throws Exception {
       if (context != null) {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/TestSupport.java

@@ -172,8 +172,6 @@ public abstract class TestSupport extends CombinationTestSupport {
 
    public static enum PersistenceAdapterChoice {LevelDB, KahaDB, AMQ, JDBC, MEM}
 
-   ;
-
    public PersistenceAdapter setDefaultPersistenceAdapter(BrokerService broker) throws IOException {
       return setPersistenceAdapter(broker, defaultPersistenceAdapter);
    }

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/broker/BrokerBenchmark.java

@@ -175,8 +175,6 @@ public class BrokerBenchmark extends BrokerTestSupport {
                   producersFinished.release();
                }
             }
-
-            ;
          }.start();
       }
 

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/broker/BrokerTestSupport.java

@@ -351,8 +351,6 @@ public class BrokerTestSupport extends CombinationTestSupport {
       }
    }
 
-   ;
-
    protected void assertNoMessagesLeft(StubConnection connection) throws InterruptedException {
       long wait = FAST_NO_MESSAGE_LEFT_ASSERT ? 0 : maxWait;
       while (true) {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/broker/MarshallingBrokerTest.java

@@ -62,8 +62,6 @@ public class MarshallingBrokerTest extends BrokerTest {
          protected void dispatch(Command command) throws InterruptedException, IOException {
             super.dispatch(wireFormat.unmarshal(wireFormat.marshal(command)));
          }
-
-         ;
       };
    }
 

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/broker/policy/MessageListenerDeadLetterTest.java

@@ -96,8 +96,6 @@ public class MessageListenerDeadLetterTest extends DeadLetterTestSupport {
       super.tearDown();
    }
 
-   ;
-
    @Override
    protected ActiveMQConnectionFactory createConnectionFactory() throws Exception {
       ActiveMQConnectionFactory answer = super.createConnectionFactory();

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/broker/region/QueueDuplicatesFromStoreTest.java

@@ -164,8 +164,6 @@ public class QueueDuplicatesFromStoreTest extends TestCase {
          public void destroy() {
          }
 
-         ;
-
          @Override
          public void gc() {
          }

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/bugs/AMQ1936Test.java

@@ -75,7 +75,6 @@ public class AMQ1936Test extends TestCase {
       broker.setDeleteAllMessagesOnStartup(true);
       broker.start();
       connectionFactory = new ActiveMQConnectionFactory("vm://test");
-      ;
    }
 
    @Override

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/bugs/AMQ3014Test.java

@@ -195,7 +195,6 @@ public class AMQ3014Test {
       }
 
       Assert.assertFalse("Timed out waiting for bridge to form.", remoteBrokerInfos.isEmpty());
-      ;
       Assert.assertNotNull("Local broker ID is null.", remoteBrokerInfos.get(0).getBrokerId());
    }
 }

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/bugs/AMQ3405Test.java

@@ -192,8 +192,6 @@ public class AMQ3405Test extends TestSupport {
       }
    }
 
-   ;
-
    @Override
    protected ActiveMQConnectionFactory createConnectionFactory() throws Exception {
       ActiveMQConnectionFactory answer = super.createConnectionFactory();

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/bugs/AMQ4083Test.java

@@ -493,7 +493,6 @@ public class AMQ4083Test {
          }
          received.acknowledge();
       }
-      ;
 
       assertEquals("got messages", messageCount + 1, messages.size());
 
@@ -501,7 +500,6 @@ public class AMQ4083Test {
       while ((received = dlqConsumer.receive(1000)) != null) {
          dlqMessages.add(received);
       }
-      ;
 
       assertEquals("got dlq messages", data.length - 1, dlqMessages.size());
 

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/bugs/AMQ4485LowLimitTest.java

@@ -246,7 +246,6 @@ public class AMQ4485LowLimitTest extends JmsMultipleBrokersTestSupport {
                   if (tally.expected.size() == 1) {
                      startConsumer(tally.brokerName, tally.destination);
                   }
-                  ;
                   return false;
                }
                LOG.info("got tally on " + tally.brokerName);

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/joramtests/ActiveMQAdmin.java

@@ -45,7 +45,6 @@ public class ActiveMQAdmin implements Admin {
          final Hashtable<String, String> env = new Hashtable<>();
          env.put("java.naming.factory.initial", "org.eclipse.jetty.jndi.InitialContextFactory");
          env.put("java.naming.factory.url.pkgs", "org.eclipse.jetty.jndi");
-         ;
          context = new InitialContext(env);
       }
       catch (NamingException e) {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/network/NetworkFailoverTest.java

@@ -92,7 +92,6 @@ public class NetworkFailoverTest extends TestCase {
                catch (JMSException e) {
                   e.printStackTrace();
                }
-               ;
             }
             catch (Exception e) {
                LOG.warn("*** Responder listener caught exception: ", e);

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/perf/NetworkedSyncTest.java

@@ -253,7 +253,6 @@ class Producer implements Runnable {
 class Consumer implements Runnable {
 
    private static final Logger LOG = LoggerFactory.getLogger(Consumer.class);
-   ;
 
    /**
     * connect to broker and receive messages

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/store/kahadb/KahaDBStoreRecoveryBrokerTest.java

@@ -46,7 +46,6 @@ public class KahaDBStoreRecoveryBrokerTest extends RecoveryBrokerTest {
 
    enum CorruptionType {None, FailToLoad, LoadInvalid, LoadCorrupt, LoadOrderIndex0}
 
-   ;
    public CorruptionType failTest = CorruptionType.None;
 
    @Override

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/store/kahadb/plist/PListTest.java

@@ -232,7 +232,6 @@ public class PListTest {
             }
          }
       }
-      ;
 
       class B implements Runnable {
 
@@ -259,7 +258,6 @@ public class PListTest {
             }
          }
       }
-      ;
 
       executor.execute(new A());
       executor.execute(new A());

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/transport/tcp/ServerSocketTstFactory.java

@@ -52,8 +52,6 @@ public class ServerSocketTstFactory extends ServerSocketFactory {
       }
    }
 
-   ;
-
    private final Random rnd;
 
    public ServerSocketTstFactory() {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/transport/tcp/SocketTstFactory.java

@@ -139,8 +139,6 @@ public class SocketTstFactory extends SocketFactory {
       }
    }
 
-   ;
-
    private final Random rnd;
 
    public SocketTstFactory() {

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/usecases/ConcurrentProducerDurableConsumerTest.java

@@ -264,7 +264,6 @@ public class ConcurrentProducerDurableConsumerTest extends TestSupport {
                   LOG.info("Signalled add consumer");
                }
             }
-            ;
             if (count % 5000 == 0) {
                LOG.info("Sent " + count + ", singleSendMax:" + max);
             }

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/usecases/ConcurrentProducerQueueConsumerTest.java

@@ -249,7 +249,6 @@ public class ConcurrentProducerQueueConsumerTest extends TestSupport {
                   LOG.info("Signalled add consumer");
                }
             }
-            ;
             if (count % 5000 == 0) {
                LOG.info("Sent " + count + ", singleSendMax:" + max);
             }

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/usecases/ExpiredMessagesTest.java

@@ -214,8 +214,6 @@ public class ExpiredMessagesTest extends CombinationTestSupport {
 
    }
 
-   ;
-
    private MessageConsumer createDlqConsumer(Connection connection) throws Exception {
       return connection.createSession(false, Session.AUTO_ACKNOWLEDGE).createConsumer(dlqDestination);
    }

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/usecases/JdbcDurableSubDupTest.java

@@ -198,7 +198,6 @@ public class JdbcDurableSubDupTest {
          }
          catch (InterruptedException e) {
          }
-         ;
 
          try {
             TextMessage m = (TextMessage) message;

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/usecases/TestBrokerConnectionDuplexExcludedDestinations.java

@@ -163,7 +163,6 @@ public class TestBrokerConnectionDuplexExcludedDestinations extends TestCase {
       // we should be able to receive excluded queue message on Hub
       MessageConsumer excludedConsumerHub = hubSession.createConsumer(excludedQueueHub);
       msg = excludedConsumerHub.receive(200);
-      ;
       assertEquals(excludedMsgHub, msg);
 
       hubProducer.close();

tests/activemq5-unit-tests/src/test/java/org/apache/activemq/util/LinkedNodeTest.java

@@ -31,8 +31,6 @@ public class LinkedNodeTest extends TestCase {
          this.v = v;
       }
 
-      ;
-
       @Override
       public String toString() {
          return "" + v;