Remove unneeded final modifiers 06/18006/1
authorRobert Varga <rovarga@cisco.com>
Thu, 9 Apr 2015 11:19:08 +0000 (13:19 +0200)
committerRobert Varga <rovarga@cisco.com>
Thu, 9 Apr 2015 11:19:08 +0000 (13:19 +0200)
Static methods are implicitly final, no need to declare them as final.

Change-Id: Ia3b085ce0e4cde46952d1b4afc6c908fdec4a506
Signed-off-by: Robert Varga <rovarga@cisco.com>
yang/yang-data-api/src/main/java/org/opendaylight/yangtools/yang/data/api/schema/tree/spi/TreeNodeFactory.java
yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/codec/AbstractIntegerStringCodec.java
yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/codec/xml/XmlDocumentUtils.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/spi/meta/StatementSupportBundle.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/spi/meta/StmtContextUtils.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/util/TextToASTTransformer.java

index 4f0a2201d7c58aa59770e80099c1b68bd4608871..601915f70ceff0fde6c383eb6594a66e479fcf49 100644 (file)
@@ -28,7 +28,7 @@ public final class TreeNodeFactory {
      * @param version data node version
      * @return new AbstractTreeNode instance, covering the data tree provided
      */
-    public static final TreeNode createTreeNodeRecursively(final NormalizedNode<?, ?> data, final Version version) {
+    public static TreeNode createTreeNodeRecursively(final NormalizedNode<?, ?> data, final Version version) {
         if (data instanceof NormalizedNodeContainer<?, ?, ?>) {
             @SuppressWarnings("unchecked")
             NormalizedNodeContainer<?, ?, NormalizedNode<?, ?>> container = (NormalizedNodeContainer<?, ?, NormalizedNode<?, ?>>) data;
@@ -51,7 +51,7 @@ public final class TreeNodeFactory {
      * @param version data node version
      * @return new AbstractTreeNode instance, covering the data tree provided
      */
-    public static final TreeNode createTreeNode(final NormalizedNode<?, ?> data, final Version version) {
+    public static TreeNode createTreeNode(final NormalizedNode<?, ?> data, final Version version) {
         if (data instanceof NormalizedNodeContainer<?, ?, ?>) {
             @SuppressWarnings("unchecked")
             NormalizedNodeContainer<?, ?, NormalizedNode<?, ?>> container = (NormalizedNodeContainer<?, ?, NormalizedNode<?, ?>>) data;
index 2bf9a35a34e6024062f71e142fe9ac247ae21009..1739330e1a7314a6a6aefafff0940ee6b5ac18f5 100644 (file)
@@ -117,7 +117,7 @@ abstract class AbstractIntegerStringCodec<N extends Number & Comparable<N>, T ex
     }
 
 
-    private final void validate(final N value) {
+    private void validate(final N value) {
         if (rangeConstraints.isEmpty()) {
             return;
         }
@@ -159,7 +159,7 @@ abstract class AbstractIntegerStringCodec<N extends Number & Comparable<N>, T ex
         return type.getRangeConstraints();
     }
 
-    private static final int provideBase(final String integer) {
+    private static int provideBase(final String integer) {
         if (integer == null) {
             throw new IllegalArgumentException("String representing integer number cannot be NULL");
         }
index 24abc3507143ae36bc8c566d50035ce078d94ee4..a4812c7316c81593bcf8869bc7919e448d6e26b7 100644 (file)
@@ -30,14 +30,13 @@ import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 
 public final class XmlDocumentUtils {
+    public static final QName OPERATION_ATTRIBUTE_QNAME = QName.create(SchemaContext.NAME, "operation");
+    private static final Logger LOG = LoggerFactory.getLogger(XmlDocumentUtils.class);
 
     private XmlDocumentUtils() {
         throw new UnsupportedOperationException("Utility class should not be instantiated");
     }
 
-    public static final QName OPERATION_ATTRIBUTE_QNAME = QName.create(SchemaContext.NAME, "operation");
-    private static final Logger LOG = LoggerFactory.getLogger(XmlDocumentUtils.class);
-
     public static Document getDocument() {
         DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
         Document doc = null;
@@ -50,7 +49,7 @@ public final class XmlDocumentUtils {
         return doc;
     }
 
-    private static final Element createElementFor(final Document doc, final QName qname, final Object obj) {
+    private static Element createElementFor(final Document doc, final QName qname, final Object obj) {
         final Element ret;
         if (qname.getNamespace() != null) {
             ret = doc.createElementNS(qname.getNamespace().toString(), qname.getLocalName());
@@ -82,8 +81,6 @@ public final class XmlDocumentUtils {
         return QName.create(namespace != null ? URI.create(namespace) : null, null, localName);
     }
 
-
-
     public static Optional<ModifyAction> getModifyOperationFromAttributes(final Element xmlElement) {
         Attr attributeNodeNS = xmlElement.getAttributeNodeNS(OPERATION_ATTRIBUTE_QNAME.getNamespace().toString(), OPERATION_ATTRIBUTE_QNAME.getLocalName());
         if(attributeNodeNS == null) {
@@ -96,8 +93,7 @@ public final class XmlDocumentUtils {
         return Optional.of(action);
     }
 
-
-    public static final Optional<DataSchemaNode> findFirstSchema(final QName qname, final Iterable<DataSchemaNode> dataSchemaNode) {
+    public static Optional<DataSchemaNode> findFirstSchema(final QName qname, final Iterable<DataSchemaNode> dataSchemaNode) {
         if (dataSchemaNode != null && qname != null) {
             for (DataSchemaNode dsn : dataSchemaNode) {
                 if (qname.isEqualWithoutRevision(dsn.getQName())) {
@@ -115,7 +111,7 @@ public final class XmlDocumentUtils {
         return Optional.absent();
     }
 
-    public static final XmlCodecProvider defaultValueCodecProvider() {
+    public static XmlCodecProvider defaultValueCodecProvider() {
         return XmlUtils.DEFAULT_XML_CODEC_PROVIDER;
     }
 }
index 504a70d69b5c14daf2d1b6671db72e9c9b9db37d..2002cf513527a53ecd52c97c8c5439b4f1a68c9e 100644 (file)
@@ -31,11 +31,11 @@ public final class StatementSupportBundle implements Immutable,NamespaceBehaviou
         this.namespaceDefinitions = namespaces;
     }
 
-    public static final Builder builder() {
+    public static Builder builder() {
         return new Builder(EMPTY);
     }
 
-    public static final Builder derivedFrom(StatementSupportBundle parent) {
+    public static Builder derivedFrom(StatementSupportBundle parent) {
         return new Builder(parent);
     }
 
index 9e3cd702d16f1d45c8343bc95ae79ba02cc4e644..1838e6effc97c288cd70b5d8f860ecd8469d37b7 100644 (file)
@@ -14,40 +14,35 @@ import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
 public final class StmtContextUtils {
 
     private static final Function<StmtContext<?, ?,?>, DeclaredStatement<?>> BUILD_DECLARED = new Function<StmtContext<?,?,?>, DeclaredStatement<?>>() {
-
         @Override
         public DeclaredStatement<?> apply(StmtContext<?,?,?> input) {
             return input.buildDeclared();
         }
-
     };
 
     private static final Function<StmtContext<?, ?,?>, EffectiveStatement<?,?>> BUILD_EFFECTIVE = new Function<StmtContext<?,?,?>, EffectiveStatement<?,?>>() {
-
         @Override
         public EffectiveStatement<?,?> apply(StmtContext<?,?,?> input) {
             return input.buildEffective();
         }
-
     };
 
     private StmtContextUtils() {
         throw new UnsupportedOperationException("Utility class");
     }
 
-
     @SuppressWarnings("unchecked")
-    public static final <D extends DeclaredStatement<?>> Function<StmtContext<?, ? extends D, ?>, D> buildDeclared() {
+    public static <D extends DeclaredStatement<?>> Function<StmtContext<?, ? extends D, ?>, D> buildDeclared() {
         return Function.class.cast(BUILD_DECLARED);
     }
 
     @SuppressWarnings("unchecked")
-    public static final <E extends EffectiveStatement<?,?>> Function<StmtContext<?, ?,? extends E>, E> buildEffective() {
+    public static <E extends EffectiveStatement<?,?>> Function<StmtContext<?, ?,? extends E>, E> buildEffective() {
         return Function.class.cast(BUILD_EFFECTIVE);
     }
 
     @SuppressWarnings("unchecked")
-    public static final <AT,DT extends DeclaredStatement<AT>> AT firstAttributeOf(Iterable<? extends StmtContext<?,?,?>> contexts, Class<DT> declaredType) {
+    public static <AT,DT extends DeclaredStatement<AT>> AT firstAttributeOf(Iterable<? extends StmtContext<?,?,?>> contexts, Class<DT> declaredType) {
         for(StmtContext<?, ?, ?> ctx : contexts) {
             if(producesDeclared(ctx,declaredType)) {
                 return (AT) ctx.getStatementArgument();
@@ -56,10 +51,7 @@ public final class StmtContextUtils {
         return null;
     }
 
-    public static final boolean producesDeclared(StmtContext<?, ?, ?> ctx, Class<? extends DeclaredStatement<?>> type) {
+    public static boolean producesDeclared(StmtContext<?, ?, ?> ctx, Class<? extends DeclaredStatement<?>> type) {
         return type.isAssignableFrom(ctx.getPublicDefinition().getDeclaredRepresentationClass());
     }
-
-
-
 }
index c808f1102e132886b18d36d34a495a48d1b8130c..b312213a81250e1338b820a0bef54a958c57fe64 100644 (file)
@@ -66,7 +66,7 @@ public final class TextToASTTransformer extends SchemaSourceTransformer<YangText
         super(provider, YangTextSchemaSource.class, consumer, ASTSchemaSource.class, TRANSFORMATION);
     }
 
-    public static final TextToASTTransformer create(final SchemaRepository provider, final SchemaSourceRegistry consumer) {
+    public static TextToASTTransformer create(final SchemaRepository provider, final SchemaSourceRegistry consumer) {
         return new TextToASTTransformer(provider, consumer);
     }
 }