Add FinalClass checkstyle check

And mark affected classes final.

config/checkstyle.xml

@@ -54,6 +54,7 @@
 	</module>
 	<module name="JavadocPackage"/>
 	<module name="TreeWalker">
+		<module name="FinalClass"/>
 		<module name="FileContentsHolder"/>
 		<module name="UnusedImports">
 			<property name="processJavadoc" value="true"/>

smack-android-extensions/src/main/java/org/jivesoftware/smackx/ping/android/ServerPingWithAlarmManager.java

@@ -59,7 +59,7 @@ import android.os.SystemClock;
  * functionality.
  * </p>
  */
-public class ServerPingWithAlarmManager extends Manager {
+public final class ServerPingWithAlarmManager extends Manager {
 
 	private static final Logger LOGGER = Logger.getLogger(ServerPingWithAlarmManager.class
 			.getName());

smack-android/src/main/java/org/jivesoftware/smack/util/stringencoder/android/AndroidBase64Encoder.java

@@ -22,7 +22,7 @@ import android.util.Base64;
  * A Base 64 encoding implementation based on android.util.Base64.
  * @author Florian Schmaus
  */
-public class AndroidBase64Encoder implements org.jivesoftware.smack.util.stringencoder.Base64.Encoder {
+public final class AndroidBase64Encoder implements org.jivesoftware.smack.util.stringencoder.Base64.Encoder {
 
     private static AndroidBase64Encoder instance = new AndroidBase64Encoder();
 

smack-android/src/main/java/org/jivesoftware/smack/util/stringencoder/android/AndroidBase64UrlSafeEncoder.java

@@ -23,7 +23,7 @@ import org.jivesoftware.smack.util.stringencoder.StringEncoder;
 
 import android.util.Base64;
 
-public class AndroidBase64UrlSafeEncoder implements StringEncoder {
+public final class AndroidBase64UrlSafeEncoder implements StringEncoder {
 
     private static AndroidBase64UrlSafeEncoder instance = new AndroidBase64UrlSafeEncoder();
 

smack-bosh/src/main/java/org/jivesoftware/smack/bosh/BOSHConfiguration.java

@@ -30,7 +30,7 @@ import org.jivesoftware.smack.proxy.ProxyInfo;
  * @see XMPPBOSHConnection
  * @author Guenther Niess
  */
-public class BOSHConfiguration extends ConnectionConfiguration {
+public final class BOSHConfiguration extends ConnectionConfiguration {
 
     private final boolean https;
     private final String file;
@@ -73,7 +73,7 @@ public class BOSHConfiguration extends ConnectionConfiguration {
         return new Builder();
     }
 
-    public static class Builder extends ConnectionConfiguration.Builder<Builder, BOSHConfiguration> {
+    public static final class Builder extends ConnectionConfiguration.Builder<Builder, BOSHConfiguration> {
         private boolean https;
         private String file;
 

smack-core/src/main/java/org/jivesoftware/smack/PacketCollector.java

@@ -271,7 +271,7 @@ public class PacketCollector {
         return new Configuration();
     }
 
-    public static class Configuration {
+    public static final class Configuration {
         private StanzaFilter packetFilter;
         private int size = SmackConfiguration.getPacketCollectorSize();
         private PacketCollector collectorToReset;

smack-core/src/main/java/org/jivesoftware/smack/ReconnectionManager.java

@@ -47,7 +47,7 @@ import java.util.logging.Logger;
  * @author Francisco Vives
  * @author Luca Stucchi
  */
-public class ReconnectionManager {
+public final class ReconnectionManager {
     private static final Logger LOGGER = Logger.getLogger(ReconnectionManager.class.getName());
 
     private static final Map<AbstractXMPPConnection, ReconnectionManager> INSTANCES = new WeakHashMap<AbstractXMPPConnection, ReconnectionManager>();

smack-core/src/main/java/org/jivesoftware/smack/SmackException.java

@@ -60,7 +60,7 @@ public class SmackException extends Exception {
      * connection instance. You can modify (e.g. increase) the stanza(/packet) reply timeout with
      * {@link XMPPConnection#setPacketReplyTimeout(long)}.
      */
-    public static class NoResponseException extends SmackException {
+    public static final class NoResponseException extends SmackException {
         /**
          * 
          */

smack-core/src/main/java/org/jivesoftware/smack/compress/packet/Compressed.java

@@ -18,7 +18,7 @@ package org.jivesoftware.smack.compress.packet;
 
 import org.jivesoftware.smack.packet.FullStreamElement;
 
-public class Compressed extends FullStreamElement {
+public final class Compressed extends FullStreamElement {
 
     public static final String ELEMENT = "compressed";
     public static final String NAMESPACE = Compress.NAMESPACE;

smack-core/src/main/java/org/jivesoftware/smack/filter/IQTypeFilter.java

@@ -27,7 +27,7 @@ import org.jivesoftware.smack.util.Objects;
  * @author Alexander Wenckus
  * 
  */
-public class IQTypeFilter extends FlexibleStanzaTypeFilter<IQ> {
+public final class IQTypeFilter extends FlexibleStanzaTypeFilter<IQ> {
 
     public static final StanzaFilter GET = new IQTypeFilter(Type.get);
     public static final StanzaFilter SET = new IQTypeFilter(Type.set);

smack-core/src/main/java/org/jivesoftware/smack/filter/MessageTypeFilter.java

@@ -27,7 +27,7 @@ import org.jivesoftware.smack.packet.Message.Type;
  * @see org.jivesoftware.smack.packet.Message.Type
  * @author Ward Harold
  */
-public class MessageTypeFilter extends FlexibleStanzaTypeFilter<Message> {
+public final class MessageTypeFilter extends FlexibleStanzaTypeFilter<Message> {
 
     public static final StanzaFilter NORMAL = new MessageTypeFilter(Type.normal);
     public static final StanzaFilter CHAT = new MessageTypeFilter(Type.chat);

smack-core/src/main/java/org/jivesoftware/smack/filter/MessageWithBodiesFilter.java

@@ -22,7 +22,7 @@ import org.jivesoftware.smack.packet.Message;
 /**
  * Filters message stanzas which have at least one body.
  */
-public class MessageWithBodiesFilter extends FlexibleStanzaTypeFilter<Message> {
+public final class MessageWithBodiesFilter extends FlexibleStanzaTypeFilter<Message> {
 
     public static final StanzaFilter INSTANCE = new MessageWithBodiesFilter();
 

smack-core/src/main/java/org/jivesoftware/smack/filter/MessageWithSubjectFilter.java

@@ -22,7 +22,7 @@ import org.jivesoftware.smack.packet.Message;
 /**
  * Filters message stanzas which have at least one body.
  */
-public class MessageWithSubjectFilter extends FlexibleStanzaTypeFilter<Message> {
+public final class MessageWithSubjectFilter extends FlexibleStanzaTypeFilter<Message> {
 
     public static final StanzaFilter INSTANCE = new MessageWithSubjectFilter();
 

smack-core/src/main/java/org/jivesoftware/smack/filter/PresenceTypeFilter.java

@@ -24,7 +24,7 @@ import org.jivesoftware.smack.util.Objects;
  * A filter for Presence types. Returns true only if the stanza is an Presence stanza(/packet) and it matches the type provided in the
  * constructor.
  */
-public class PresenceTypeFilter extends FlexibleStanzaTypeFilter<Presence> {
+public final class PresenceTypeFilter extends FlexibleStanzaTypeFilter<Presence> {
 
     public static final PresenceTypeFilter AVAILABLE = new PresenceTypeFilter(Type.available);
     public static final PresenceTypeFilter UNAVAILABLE = new PresenceTypeFilter(Type.unavailable);

smack-core/src/main/java/org/jivesoftware/smack/packet/Bind.java

@@ -70,7 +70,7 @@ public class Bind extends IQ {
         return xml;
     }
 
-    public static class Feature implements ExtensionElement {
+    public static final class Feature implements ExtensionElement {
 
         public static final Feature INSTANCE = new Feature();
 

smack-core/src/main/java/org/jivesoftware/smack/packet/Message.java

@@ -481,7 +481,7 @@ public final class Message extends Stanza implements TypedCloneable<Message> {
     /**
      * Represents a message subject, its language and the content of the subject.
      */
-    public static class Subject {
+    public static final class Subject {
 
         private final String subject;
         private final String language;
@@ -544,7 +544,7 @@ public final class Message extends Stanza implements TypedCloneable<Message> {
     /**
      * Represents a message body, its language and the content of the message.
      */
-    public static class Body {
+    public static final class Body {
 
         private final String message;
         private final String language;

smack-core/src/main/java/org/jivesoftware/smack/util/MultiMap.java

@@ -208,7 +208,7 @@ public class MultiMap<K,V> {
         return entrySet;
     }
 
-    private static class SimpleMapEntry<K, V> implements Map.Entry<K, V> {
+    private static final class SimpleMapEntry<K, V> implements Map.Entry<K, V> {
 
         private final K key;
         private V value;

smack-core/src/test/java/org/jivesoftware/smack/DummyConnection.java

@@ -223,7 +223,7 @@ public class DummyConnection extends AbstractXMPPConnection {
             return new Builder();
         }
 
-        public static class Builder
+        public static final class Builder
                         extends
                         ConnectionConfiguration.Builder<Builder, DummyConnectionConfiguration> {
 

smack-debug/src/main/java/org/jivesoftware/smackx/debugger/EnhancedDebuggerWindow.java

@@ -56,7 +56,7 @@ import org.jivesoftware.smack.provider.ProviderManager;
  *
  * @author Gaston Dombiak
  */
-public class EnhancedDebuggerWindow {
+public final class EnhancedDebuggerWindow {
 
     private static EnhancedDebuggerWindow instance;
 

smack-experimental/src/main/java/org/jivesoftware/smackx/carbons/CarbonManager.java

@@ -47,7 +47,7 @@ import org.jivesoftware.smackx.disco.ServiceDiscoveryManager;
  *
  * @author Georg Lukas
  */
-public class CarbonManager extends Manager {
+public final class CarbonManager extends Manager {
 
     private static Map<XMPPConnection, CarbonManager> INSTANCES = new WeakHashMap<XMPPConnection, CarbonManager>();
 

smack-experimental/src/main/java/org/jivesoftware/smackx/carbons/packet/CarbonExtension.java

@@ -131,7 +131,7 @@ public class CarbonExtension implements ExtensionElement {
      * Stanza(/Packet) extension indicating that a message may not be carbon-copied.  Adding this
      * extension to any message will disallow that message from being copied. 
      */
-    public static class Private implements ExtensionElement {
+    public static final class Private implements ExtensionElement {
         public static final Private INSTANCE = new Private();
         public static final String ELEMENT = "private";
 

smack-experimental/src/main/java/org/jivesoftware/smackx/csi/packet/ClientStateIndication.java

@@ -27,7 +27,7 @@ import org.jivesoftware.smack.packet.ExtensionElement;
 public class ClientStateIndication {
     public static final String NAMESPACE = "urn:xmpp:csi:0";
 
-    public static class Active extends FullStreamElement {
+    public static final class Active extends FullStreamElement {
         public static final Active INSTANCE = new Active();
         public static final String ELEMENT = "active";
 
@@ -50,7 +50,7 @@ public class ClientStateIndication {
         }
     }
 
-    public static class Inactive extends FullStreamElement {
+    public static final class Inactive extends FullStreamElement {
         public static final Inactive INSTANCE = new Inactive();
         public static final String ELEMENT = "inactive";
 
@@ -73,7 +73,7 @@ public class ClientStateIndication {
         }
     }
 
-    public static class Feature implements ExtensionElement {
+    public static final class Feature implements ExtensionElement {
         public static final Feature INSTANCE = new Feature();
         public static final String ELEMENT = "csi";
 

smack-extensions/src/main/java/org/jivesoftware/smackx/address/packet/MultipleAddresses.java

@@ -128,7 +128,7 @@ public class MultipleAddresses implements ExtensionElement {
         return buf;
     }
 
-    public static class Address implements NamedElement {
+    public static final class Address implements NamedElement {
 
         public static final String ELEMENT = "address";
 

smack-extensions/src/main/java/org/jivesoftware/smackx/bookmarks/BookmarkManager.java

@@ -40,7 +40,7 @@ import org.jivesoftware.smackx.iqprivate.PrivateDataManager;
  *
  * @author Alexander Wenckus
  */
-public class BookmarkManager {
+public final class BookmarkManager {
     private static final Map<XMPPConnection, BookmarkManager> bookmarkManagerMap = new WeakHashMap<XMPPConnection, BookmarkManager>();
 
     static {

smack-extensions/src/main/java/org/jivesoftware/smackx/bytestreams/ibb/InBandBytestreamManager.java

@@ -79,7 +79,7 @@ import org.jxmpp.jid.Jid;
  * 
  * @author Henning Staib
  */
-public class InBandBytestreamManager implements BytestreamManager {
+public final class InBandBytestreamManager implements BytestreamManager {
 
     /**
      * Stanzas that can be used to encapsulate In-Band Bytestream data packets.

smack-extensions/src/main/java/org/jivesoftware/smackx/bytestreams/socks5/Socks5Proxy.java

@@ -68,7 +68,7 @@ import org.jivesoftware.smack.SmackException;
  * 
  * @author Henning Staib
  */
-public class Socks5Proxy {
+public final class Socks5Proxy {
     private static final Logger LOGGER = Logger.getLogger(Socks5Proxy.class.getName());
 
     /* SOCKS5 proxy singleton */

smack-extensions/src/main/java/org/jivesoftware/smackx/caps/EntityCapsManager.java

@@ -71,7 +71,7 @@ import java.security.NoSuchAlgorithmException;
  * @author Florian Schmaus
  * @see <a href="http://www.xmpp.org/extensions/xep-0115.html">XEP-0115: Entity Capabilities</a>
  */
-public class EntityCapsManager extends Manager {
+public final class EntityCapsManager extends Manager {
     private static final Logger LOGGER = Logger.getLogger(EntityCapsManager.class.getName());
 
     public static final String NAMESPACE = CapsExtension.NAMESPACE;

smack-extensions/src/main/java/org/jivesoftware/smackx/chatstates/ChatStateManager.java

@@ -49,7 +49,7 @@ import org.jivesoftware.smackx.disco.ServiceDiscoveryManager;
  * @see org.jivesoftware.smackx.chatstates.ChatState
  * @see org.jivesoftware.smackx.chatstates.packet.ChatStateExtension
  */
-public class ChatStateManager extends Manager {
+public final class ChatStateManager extends Manager {
     public static final String NAMESPACE = "http://jabber.org/protocol/chatstates";
 
     private static final Map<XMPPConnection, ChatStateManager> INSTANCES =

smack-extensions/src/main/java/org/jivesoftware/smackx/commands/AdHocCommandManager.java

@@ -60,7 +60,7 @@ import org.jxmpp.jid.Jid;
  * 
  * @author Gabriel Guardincerri
  */
-public class AdHocCommandManager extends Manager {
+public final class AdHocCommandManager extends Manager {
     public static final String NAMESPACE = "http://jabber.org/protocol/commands";
 
     private static final Logger LOGGER = Logger.getLogger(AdHocCommandManager.class.getName());

smack-extensions/src/main/java/org/jivesoftware/smackx/delay/filter/DelayedStanzaFilter.java

@@ -24,7 +24,7 @@ import org.jivesoftware.smackx.delay.DelayInformationManager;
 /**
  * Filters stanza with delay information, ie. stanzas that got delayed for some reason
  */
-public class DelayedStanzaFilter implements StanzaFilter {
+public final class DelayedStanzaFilter implements StanzaFilter {
 
     public static final StanzaFilter INSTANCE = new DelayedStanzaFilter();
 

smack-extensions/src/main/java/org/jivesoftware/smackx/disco/ServiceDiscoveryManager.java

@@ -63,7 +63,7 @@ import java.util.logging.Logger;
  * 
  * @author Gaston Dombiak
  */
-public class ServiceDiscoveryManager extends Manager {
+public final class ServiceDiscoveryManager extends Manager {
 
     private static final Logger LOGGER = Logger.getLogger(ServiceDiscoveryManager.class.getName());
 

smack-extensions/src/main/java/org/jivesoftware/smackx/filetransfer/FileTransferManager.java

@@ -43,7 +43,7 @@ import java.util.concurrent.CopyOnWriteArrayList;
  * @author Alexander Wenckus
  * 
  */
-public class FileTransferManager extends Manager {
+public final class FileTransferManager extends Manager {
 
     private static final Map<XMPPConnection, FileTransferManager> INSTANCES = new WeakHashMap<XMPPConnection, FileTransferManager>();
 

smack-extensions/src/main/java/org/jivesoftware/smackx/filetransfer/FileTransferNegotiator.java

@@ -52,7 +52,7 @@ import org.jxmpp.jid.Jid;
  * @author Alexander Wenckus
  * @see <a href="http://xmpp.org/extensions/xep-0096.html">XEP-0096: SI File Transfer</a>
  */
-public class FileTransferNegotiator extends Manager {
+public final class FileTransferNegotiator extends Manager {
 
     public static final String SI_NAMESPACE = "http://jabber.org/protocol/si";
     public static final String SI_PROFILE_FILE_TRANSFER_NAMESPACE = "http://jabber.org/protocol/si/profile/file-transfer";

smack-extensions/src/main/java/org/jivesoftware/smackx/filetransfer/IBBTransferNegotiator.java

@@ -105,7 +105,7 @@ public class IBBTransferNegotiator extends StreamNegotiator {
     /**
      * Derive from InBandBytestreamRequest to access protected constructor.
      */
-    private static class ByteStreamRequest extends InBandBytestreamRequest {
+    private static final class ByteStreamRequest extends InBandBytestreamRequest {
 
         private ByteStreamRequest(InBandBytestreamManager manager, Open byteStreamRequest) {
             super(manager, byteStreamRequest);

smack-extensions/src/main/java/org/jivesoftware/smackx/filetransfer/Socks5TransferNegotiator.java

@@ -119,7 +119,7 @@ public class Socks5TransferNegotiator extends StreamNegotiator {
     /**
      * Derive from Socks5BytestreamRequest to access protected constructor.
      */
-    private static class ByteStreamRequest extends Socks5BytestreamRequest {
+    private static final class ByteStreamRequest extends Socks5BytestreamRequest {
 
         private ByteStreamRequest(Socks5BytestreamManager manager, Bytestream byteStreamRequest) {
             super(manager, byteStreamRequest);

smack-extensions/src/main/java/org/jivesoftware/smackx/geoloc/GeoLocationManager.java

@@ -28,7 +28,7 @@ import org.jivesoftware.smack.packet.Message;
 import org.jivesoftware.smackx.geoloc.packet.GeoLocation;
 import org.jxmpp.jid.Jid;
 
-public class GeoLocationManager extends Manager {
+public final class GeoLocationManager extends Manager {
 
     private static final Map<XMPPConnection, GeoLocationManager> INSTANCES = new WeakHashMap<XMPPConnection, GeoLocationManager>();
 

smack-extensions/src/main/java/org/jivesoftware/smackx/geoloc/packet/GeoLocation.java

@@ -33,7 +33,7 @@ import org.jivesoftware.smack.util.XmlStringBuilder;
  * @see <a href="http://www.xmpp.org/extensions/xep-0080.html">XEP-0080</a>
  * @author Ishan Khanna
  */
-public class GeoLocation implements Serializable, ExtensionElement {
+public final class GeoLocation implements Serializable, ExtensionElement {
 
     private static final long serialVersionUID = 1L;
     public static final String NAMESPACE = "http://jabber.org/protocol/geoloc";

smack-extensions/src/main/java/org/jivesoftware/smackx/iqlast/LastActivityManager.java

@@ -90,7 +90,7 @@ import org.jxmpp.jid.Jid;
  *      Activity</a>
  */
 
-public class LastActivityManager extends Manager {
+public final class LastActivityManager extends Manager {
     private static final Map<XMPPConnection, LastActivityManager> instances = new WeakHashMap<XMPPConnection, LastActivityManager>();
 //    private static final PacketFilter IQ_GET_LAST_FILTER = new AndFilter(IQTypeFilter.GET,
 //                    new StanzaTypeFilter(LastActivity.class));

smack-extensions/src/main/java/org/jivesoftware/smackx/iqprivate/PrivateDataManager.java

@@ -59,7 +59,7 @@ import java.util.WeakHashMap;
  *
  * @author Matt Tucker
  */
-public class PrivateDataManager extends Manager {
+public final class PrivateDataManager extends Manager {
     private static final Map<XMPPConnection, PrivateDataManager> instances = new WeakHashMap<XMPPConnection, PrivateDataManager>();
 
     public static synchronized PrivateDataManager getInstanceFor(XMPPConnection connection) {

smack-extensions/src/main/java/org/jivesoftware/smackx/iqregister/AccountManager.java

@@ -41,7 +41,7 @@ import org.jivesoftware.smackx.iqregister.packet.Registration;
  *
  * @author Matt Tucker
  */
-public class AccountManager extends Manager {
+public final class AccountManager extends Manager {
 
     private static final Logger LOGGER = Logger.getLogger(AccountManager.class.getName());
 

smack-extensions/src/main/java/org/jivesoftware/smackx/iqregister/packet/Registration.java

@@ -100,7 +100,7 @@ public class Registration extends IQ {
         return xml;
     }
 
-    public static class Feature implements ExtensionElement {
+    public static final class Feature implements ExtensionElement {
 
         public static final String ELEMENT = "register";
         public static final String NAMESPACE = "http://jabber.org/features/iq-register";

smack-extensions/src/main/java/org/jivesoftware/smackx/iqversion/VersionManager.java

@@ -53,7 +53,7 @@ import org.jxmpp.jid.Jid;
  *
  * @author Georg Lukas
  */
-public class VersionManager extends Manager {
+public final class VersionManager extends Manager {
     private static final Map<XMPPConnection, VersionManager> INSTANCES = new WeakHashMap<XMPPConnection, VersionManager>();
 
     private static Version defaultVersion;

smack-extensions/src/main/java/org/jivesoftware/smackx/muc/MultiUserChatManager.java

@@ -55,7 +55,7 @@ import org.jxmpp.jid.DomainBareJid;
 import org.jxmpp.jid.Jid;
 import org.jxmpp.jid.JidWithLocalpart;
 
-public class MultiUserChatManager extends Manager {
+public final class MultiUserChatManager extends Manager {
     private final static String DISCO_NODE = MUCInitialPresence.NAMESPACE + "#rooms";
 
     private static final Logger LOGGER = Logger.getLogger(MultiUserChatManager.class.getName());

smack-extensions/src/main/java/org/jivesoftware/smackx/muc/packet/MUCUser.java

@@ -413,7 +413,7 @@ public class MUCUser implements ExtensionElement {
      *
      * @author Gaston Dombiak
      */
-    public static class Status implements NamedElement {
+    public static final class Status implements NamedElement {
         public static final String ELEMENT = "status";
 
         private static final Map<Integer, Status> statusMap = new HashMap<Integer, Status>(8);

smack-extensions/src/main/java/org/jivesoftware/smackx/ping/PingManager.java

@@ -60,7 +60,7 @@ import org.jxmpp.jid.Jid;
  * @author Florian Schmaus
  * @see <a href="http://www.xmpp.org/extensions/xep-0199.html">XEP-0199:XMPP Ping</a>
  */
-public class PingManager extends Manager {
+public final class PingManager extends Manager {
     private static final Logger LOGGER = Logger.getLogger(PingManager.class.getName());
 
     private static final Map<XMPPConnection, PingManager> INSTANCES = new WeakHashMap<XMPPConnection, PingManager>();

smack-extensions/src/main/java/org/jivesoftware/smackx/privacy/PrivacyListManager.java

@@ -62,7 +62,7 @@ import org.jivesoftware.smackx.privacy.packet.PrivacyItem;
  * @author Francisco Vives
  * @see <a href="http://xmpp.org/extensions/xep-0016.html">XEP-16: Privacy Lists</a>
  */
-public class PrivacyListManager extends Manager {
+public final class PrivacyListManager extends Manager {
     public static final String NAMESPACE = Privacy.NAMESPACE;
 
     public static final StanzaFilter PRIVACY_FILTER = new StanzaTypeFilter(Privacy.class);

smack-extensions/src/main/java/org/jivesoftware/smackx/privacy/filter/SetActiveListFilter.java

@@ -20,7 +20,7 @@ import org.jivesoftware.smack.filter.FlexibleStanzaTypeFilter;
 import org.jivesoftware.smack.packet.IQ.Type;
 import org.jivesoftware.smackx.privacy.packet.Privacy;
 
-public class SetActiveListFilter extends FlexibleStanzaTypeFilter<Privacy> {
+public final class SetActiveListFilter extends FlexibleStanzaTypeFilter<Privacy> {
 
     public static final SetActiveListFilter INSTANCE = new SetActiveListFilter();
 

smack-extensions/src/main/java/org/jivesoftware/smackx/privacy/filter/SetDefaultListFilter.java

@@ -20,7 +20,7 @@ import org.jivesoftware.smack.filter.FlexibleStanzaTypeFilter;
 import org.jivesoftware.smack.packet.IQ.Type;
 import org.jivesoftware.smackx.privacy.packet.Privacy;
 
-public class SetDefaultListFilter extends FlexibleStanzaTypeFilter<Privacy> {
+public final class SetDefaultListFilter extends FlexibleStanzaTypeFilter<Privacy> {
 
     public static final SetDefaultListFilter INSTANCE = new SetDefaultListFilter();
 

smack-extensions/src/main/java/org/jivesoftware/smackx/receipts/DeliveryReceiptManager.java

@@ -66,7 +66,7 @@ import org.jxmpp.jid.Jid;
  * @author Georg Lukas
  * @see <a href="http://xmpp.org/extensions/xep-0184.html">XEP-0184: Message Delivery Receipts</a>
  */
-public class DeliveryReceiptManager extends Manager {
+public final class DeliveryReceiptManager extends Manager {
 
     private static final StanzaFilter MESSAGES_WITH_DEVLIERY_RECEIPT_REQUEST = new AndFilter(StanzaTypeFilter.MESSAGE,
                     new StanzaExtensionFilter(new DeliveryReceiptRequest()));

smack-extensions/src/main/java/org/jivesoftware/smackx/time/EntityTimeManager.java

@@ -36,7 +36,7 @@ import org.jivesoftware.smackx.disco.ServiceDiscoveryManager;
 import org.jivesoftware.smackx.time.packet.Time;
 import org.jxmpp.jid.Jid;
 
-public class EntityTimeManager extends Manager {
+public final class EntityTimeManager extends Manager {
 
     private static final Map<XMPPConnection, EntityTimeManager> INSTANCES = new WeakHashMap<XMPPConnection, EntityTimeManager>();
 

smack-extensions/src/main/java/org/jivesoftware/smackx/vcardtemp/VCardManager.java

@@ -33,7 +33,7 @@ import org.jivesoftware.smackx.vcardtemp.packet.VCard;
 import org.jxmpp.jid.BareJid;
 import org.jxmpp.jid.Jid;
 
-public class VCardManager extends Manager {
+public final class VCardManager extends Manager {
     public static final String NAMESPACE = VCard.NAMESPACE;
     public static final String ELEMENT = VCard.ELEMENT;
 

smack-extensions/src/main/java/org/jivesoftware/smackx/xdata/XDataManager.java

@@ -30,7 +30,7 @@ import org.jivesoftware.smackx.disco.ServiceDiscoveryManager;
 import org.jivesoftware.smackx.xdata.packet.DataForm;
 import org.jxmpp.jid.Jid;
 
-public class XDataManager extends Manager {
+public final class XDataManager extends Manager {
 
     /**
      * The value of {@link DataForm#NAMESPACE}.

smack-extensions/src/test/java/org/jivesoftware/smackx/bytestreams/socks5/Socks5TestProxy.java

@@ -37,7 +37,7 @@ import org.jivesoftware.smack.SmackException;
  * 
  * @author Henning Staib
  */
-public class Socks5TestProxy {
+public final class Socks5TestProxy {
     private static final Logger LOGGER = Logger.getLogger(Socks5TestProxy.class.getName());
 
     /* SOCKS5 proxy singleton */

smack-im/src/main/java/org/jivesoftware/smack/chat/ChatManager.java

@@ -53,7 +53,7 @@ import org.jxmpp.jid.JidWithLocalpart;
  *
  * @author Alexander Wenckus
  */
-public class ChatManager extends Manager{
+public final class ChatManager extends Manager{
 
     private static final Logger LOGGER = Logger.getLogger(ChatManager.class.getName());
 

smack-im/src/main/java/org/jivesoftware/smack/roster/Roster.java

@@ -82,7 +82,7 @@ import org.jxmpp.jid.parts.Resourcepart;
  * @author Matt Tucker
  * @see #getInstanceFor(XMPPConnection)
  */
-public class Roster extends Manager {
+public final class Roster extends Manager {
 
     private static final Logger LOGGER = Logger.getLogger(Roster.class.getName());
 
@@ -1459,7 +1459,7 @@ public class Roster extends Manager {
     /**
      * Listens for all roster pushes and processes them.
      */
-    private class RosterPushListener extends AbstractIqRequestHandler {
+    private final class RosterPushListener extends AbstractIqRequestHandler {
 
         private RosterPushListener() {
             super(RosterPacket.ELEMENT, RosterPacket.NAMESPACE, Type.set, Mode.sync);

smack-im/src/main/java/org/jivesoftware/smack/roster/packet/RosterVer.java

@@ -19,7 +19,7 @@ package org.jivesoftware.smack.roster.packet;
 import org.jivesoftware.smack.packet.ExtensionElement;
 import org.jivesoftware.smack.util.XmlStringBuilder;
 
-public class RosterVer implements ExtensionElement {
+public final class RosterVer implements ExtensionElement {
 
     public static final String ELEMENT = "ver";
     public static final String NAMESPACE = "urn:xmpp:features:rosterver";

smack-im/src/main/java/org/jivesoftware/smack/roster/packet/SubscriptionPreApproval.java

@@ -19,7 +19,7 @@ package org.jivesoftware.smack.roster.packet;
 import org.jivesoftware.smack.packet.ExtensionElement;
 import org.jivesoftware.smack.util.XmlStringBuilder;
 
-public class SubscriptionPreApproval implements ExtensionElement {
+public final class SubscriptionPreApproval implements ExtensionElement {
 
     public static final String ELEMENT = "sub";
     public static final String NAMESPACE = "urn:xmpp:features:pre-approval";

smack-im/src/main/java/org/jivesoftware/smack/roster/rosterstore/DirectoryRosterStore.java

@@ -44,7 +44,7 @@ import org.xmlpull.v1.XmlPullParserException;
  * @author Lars Noschinski
  * @author Fabian Schuetz
  */
-public class DirectoryRosterStore implements RosterStore {
+public final class DirectoryRosterStore implements RosterStore {
 
     private final File fileDir;
 

smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/Configuration.java

@@ -33,7 +33,7 @@ import org.jxmpp.jid.DomainBareJid;
 import org.jxmpp.jid.impl.JidCreate;
 import org.jxmpp.stringprep.XmppStringprepException;
 
-public class Configuration {
+public final class Configuration {
 
     public final DomainBareJid service;
 
@@ -92,7 +92,7 @@ public class Configuration {
         return new Builder();
     }
 
-    public static class Builder {
+    public static final class Builder {
 
         private DomainBareJid service;
 

smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/IntTestUtil.java

@@ -55,7 +55,7 @@ public class IntTestUtil {
         return new UsernameAndPassword(username, password);
     }
 
-    public static class UsernameAndPassword {
+    public static final class UsernameAndPassword {
         public final String username;
         public final String password;
 

smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/SmackIntegrationTestFramework.java

@@ -595,7 +595,7 @@ public class SmackIntegrationTestFramework {
         return (Exception) e;
     }
 
-    public static class TestRunResult {
+    public static final class TestRunResult {
         public final String testRunId = StringUtils.randomString(5);
         private final List<SuccessfulTest> successfulTests = Collections.synchronizedList(new LinkedList<SuccessfulTest>());
         private final List<FailedTest> failedIntegrationTests = Collections.synchronizedList(new LinkedList<FailedTest>());

smack-java7/src/main/java/org/jivesoftware/smack/util/stringencoder/java7/Base64.java

@@ -29,7 +29,7 @@ import java.util.logging.Logger;
  * @author rob@iharder.net
  * @version 2.2.1
  */
-public class Base64
+public final class Base64
 {
     private static final Logger LOGGER = Logger.getLogger(Base64.class.getName());
 

smack-java7/src/main/java/org/jivesoftware/smack/util/stringencoder/java7/Java7Base64Encoder.java

@@ -24,7 +24,7 @@ import org.jivesoftware.smack.util.StringUtils;
  * A Base 64 encoding implementation.
  * @author Florian Schmaus
  */
-public class Java7Base64Encoder implements org.jivesoftware.smack.util.stringencoder.Base64.Encoder {
+public final class Java7Base64Encoder implements org.jivesoftware.smack.util.stringencoder.Base64.Encoder {
 
     private static Java7Base64Encoder instance = new Java7Base64Encoder();
 

smack-java7/src/main/java/org/jivesoftware/smack/util/stringencoder/java7/Java7Base64UrlSafeEncoder.java

@@ -30,7 +30,7 @@ import org.jivesoftware.smack.util.stringencoder.StringEncoder;
  * 
  * @author Robin Collier
  */
-public class Java7Base64UrlSafeEncoder implements StringEncoder {
+public final class Java7Base64UrlSafeEncoder implements StringEncoder {
 
     private static Java7Base64UrlSafeEncoder instance = new Java7Base64UrlSafeEncoder();
 

smack-legacy/src/main/java/org/jivesoftware/smackx/workgroup/agent/WorkgroupQueue.java

@@ -170,7 +170,7 @@ public class WorkgroupQueue {
      *          chat requests.
      * </ul>
      */
-    public static class Status {
+    public static final class Status {
 
         /**
          * The queue is active and accepting new chat requests.

smack-legacy/src/main/java/org/jivesoftware/smackx/workgroup/packet/QueueDetails.java

@@ -37,7 +37,7 @@ import java.util.logging.Logger;
  * Queue details stanza(/packet) extension, which contains details about the users
  * currently in a queue.
  */
-public class QueueDetails implements ExtensionElement {
+public final class QueueDetails implements ExtensionElement {
     private static final Logger LOGGER = Logger.getLogger(QueueDetails.class.getName());
 
     /**

smack-legacy/src/main/java/org/jivesoftware/smackx/xevent/MessageEventManager.java

@@ -48,7 +48,7 @@ import org.jxmpp.jid.Jid;
  * @author Gaston Dombiak
  * @see <a href="http://xmpp.org/extensions/xep-0022.html">XEP-22: Message Events</a>
  */
-public class MessageEventManager extends Manager {
+public final class MessageEventManager extends Manager {
     private static final Logger LOGGER = Logger.getLogger(MessageEventManager.class.getName());
 
     private static final Map<XMPPConnection, MessageEventManager> INSTANCES = new WeakHashMap<>();

smack-tcp/src/main/java/org/jivesoftware/smack/sm/packet/StreamManagement.java

@@ -24,7 +24,7 @@ import org.jivesoftware.smack.util.XmlStringBuilder;
 public class StreamManagement {
     public static final String NAMESPACE = "urn:xmpp:sm:3";
 
-    public static class StreamManagementFeature implements ExtensionElement {
+    public static final class StreamManagementFeature implements ExtensionElement {
 
         public static final String ELEMENT = "sm";
         public static final StreamManagementFeature INSTANCE = new StreamManagementFeature();
@@ -320,7 +320,7 @@ public class StreamManagement {
         }
     }
 
-    public static class AckRequest extends FullStreamElement {
+    public static final class AckRequest extends FullStreamElement {
         public static final String ELEMENT = "r";
         public static final AckRequest INSTANCE = new AckRequest();
 

smack-tcp/src/main/java/org/jivesoftware/smack/sm/predicates/ForEveryMessage.java

@@ -20,7 +20,7 @@ import org.jivesoftware.smack.filter.StanzaFilter;
 import org.jivesoftware.smack.packet.Message;
 import org.jivesoftware.smack.packet.Stanza;
 
-public class ForEveryMessage implements StanzaFilter {
+public final class ForEveryMessage implements StanzaFilter {
 
     public static final ForEveryMessage INSTANCE = new ForEveryMessage();
 

smack-tcp/src/main/java/org/jivesoftware/smack/sm/predicates/ForEveryStanza.java

@@ -19,7 +19,7 @@ package org.jivesoftware.smack.sm.predicates;
 import org.jivesoftware.smack.filter.StanzaFilter;
 import org.jivesoftware.smack.packet.Stanza;
 
-public class ForEveryStanza implements StanzaFilter {
+public final class ForEveryStanza implements StanzaFilter {
 
     public static final ForEveryStanza INSTANCE = new ForEveryStanza();
 

smack-tcp/src/main/java/org/jivesoftware/smack/sm/predicates/OnceForThisStanza.java

@@ -21,7 +21,7 @@ import org.jivesoftware.smack.packet.Stanza;
 import org.jivesoftware.smack.tcp.XMPPTCPConnection;
 import org.jivesoftware.smack.util.StringUtils;
 
-public class OnceForThisStanza implements StanzaFilter {
+public final class OnceForThisStanza implements StanzaFilter {
 
     private final String id;
     private final XMPPTCPConnection connection;

smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnectionConfiguration.java

@@ -33,7 +33,7 @@ import org.jivesoftware.smack.ConnectionConfiguration;
  * }
  * </pre>
  */
-public class XMPPTCPConnectionConfiguration extends ConnectionConfiguration {
+public final class XMPPTCPConnectionConfiguration extends ConnectionConfiguration {
 
     /**
      * The default connect timeout in milliseconds. Preinitialized with 30000 (30 seconds). If this value is changed,
@@ -84,7 +84,7 @@ public class XMPPTCPConnectionConfiguration extends ConnectionConfiguration {
      * A configuration builder for XMPP connections over TCP. Use {@link XMPPTCPConnectionConfiguration#builder()} to
      * obtain a new instance and {@link #build} to build the configuration.
      */
-    public static class Builder extends ConnectionConfiguration.Builder<Builder, XMPPTCPConnectionConfiguration> {
+    public static final class Builder extends ConnectionConfiguration.Builder<Builder, XMPPTCPConnectionConfiguration> {
         private boolean compressionEnabled = false;
         private int connectTimeout = DEFAULT_CONNECT_TIMEOUT;