Use QNameModule#intern() 96/31896/3
authorRobert Varga <robert.varga@pantheon.sk>
Sat, 26 Dec 2015 16:07:30 +0000 (17:07 +0100)
committerGerrit Code Review <gerrit@opendaylight.org>
Mon, 4 Jan 2016 09:06:06 +0000 (09:06 +0000)
QNameModule#cachedReference() is deprecated, use #intern() instead.

Change-Id: Ic60eb4f3968f3e3251e2b798c385043a717c5089
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>
yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/YangConstants.java
yang/yang-data-transform/src/main/java/org/opendaylight/yangtools/transform/QNameModuleReplacementFunction.java
yang/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/Rfc6020Mapping.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/BaseTypes.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/ModuleBuilder.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/ModuleImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/impl/YangParserListenerImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/ModuleStatementSupport.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/Utils.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/ModuleEffectiveStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/SubmoduleEffectiveStatementImpl.java

index dbc25eb5b9552be1d0e728bdf8c7aba470f8f171..5adc588e0f04a0fb51c23894350b78209aa6bd80 100644 (file)
@@ -26,12 +26,12 @@ public final class YangConstants {
     /**
      * Base QNameModule for all YANG statements.
      */
-    public static final QNameModule RFC6020_YANG_MODULE = QNameModule.cachedReference(QNameModule.create(RFC6020_YANG_NAMESPACE, null));
+    public static final QNameModule RFC6020_YANG_MODULE = QNameModule.create(RFC6020_YANG_NAMESPACE, null).intern();
 
     /**
      * Base QNameModule for all YIN statements.
      */
-    public static final QNameModule RFC6020_YIN_MODULE = QNameModule.cachedReference(QNameModule.create(RFC6020_YIN_NAMESPACE, null));
+    public static final QNameModule RFC6020_YIN_MODULE = QNameModule.create(RFC6020_YIN_NAMESPACE, null).intern();
 
     private YangConstants() {
         throw new UnsupportedOperationException("Utility class");
index 6790fc004d8cbc2c1121fcd6ada502077bd2af5f..42fbccb5532da24befcdf899dd03bdc2bc0afd70 100644 (file)
@@ -25,7 +25,7 @@ class QNameModuleReplacementFunction implements Function<QName, QName> {
     public QName apply(QName input) {
         QNameModule potential = mapping.get(input.getModule());
         if (potential != null) {
-            return QName.cachedReference(QName.create(potential, input.getLocalName()));
+            return QName.create(potential, input.getLocalName()).intern();
         }
         return input;
     }
index feb35e357324c964c9a4c42857fefc2c72c4973e..d791ab960e1cdfb331696362c6927a92ee7aba0a 100644 (file)
@@ -187,7 +187,7 @@ public enum Rfc6020Mapping implements StatementDefinition {
     }
 
     @Nonnull private static QName yinQName(String nameStr) {
-        return QName.cachedReference(QName.create(YangConstants.RFC6020_YIN_MODULE, nameStr));
+        return QName.create(YangConstants.RFC6020_YIN_MODULE, nameStr).intern();
     }
 
     @Override
index dfc5dfb5c4f0ec385901000ea76b23f1c0f55cc1..facfe3080cbdbb1c545c97258d8f84454fa6d185 100644 (file)
@@ -89,7 +89,7 @@ public final class BaseTypes {
      * @return built-in base yang type QName.
      */
     public static QName constructQName(final String typeName) {
-        return QName.cachedReference(QName.create(YangConstants.RFC6020_YANG_MODULE, typeName));
+        return QName.create(YangConstants.RFC6020_YANG_MODULE, typeName).intern();
     }
 
     /**
index 07e1c9b6058dd58995e4a71618eccec742a3bab6..14ee3961e0aeef01d402692c7616afb7cb45fc33 100644 (file)
@@ -63,7 +63,7 @@ import org.opendaylight.yangtools.yang.parser.util.YangParseException;
  * otherwise result may not be valid.
  */
 public class ModuleBuilder extends AbstractDocumentedDataNodeContainerBuilder implements DocumentedNodeBuilder {
-    private static final QNameModule EMPTY_QNAME_MODULE = QNameModule.cachedReference(QNameModule.create(null, null));
+    private static final QNameModule EMPTY_QNAME_MODULE = QNameModule.create(null, null).intern();
     private static final String GROUPING_STR = "Grouping";
     private static final String TYPEDEF_STR = "typedef";
     private ModuleImpl instance;
@@ -349,7 +349,7 @@ public class ModuleBuilder extends AbstractDocumentedDataNodeContainerBuilder im
     }
 
     public void setNamespace(final URI namespace) {
-        this.qnameModule = QNameModule.cachedReference(QNameModule.create(namespace, qnameModule.getRevision()));
+        this.qnameModule = QNameModule.create(namespace, qnameModule.getRevision()).intern();
     }
 
     public String getPrefix() {
@@ -411,7 +411,7 @@ public class ModuleBuilder extends AbstractDocumentedDataNodeContainerBuilder im
     }
 
     public void setRevision(final Date revision) {
-        this.qnameModule = QNameModule.cachedReference(QNameModule.create(qnameModule.getNamespace(), revision));
+        this.qnameModule = QNameModule.create(qnameModule.getNamespace(), revision).intern();
     }
 
     public void setPrefix(final String prefix) {
index bfd515b766c7e118f54b31c83d226a048462a143..9b04c6875c8fdd624b89b9dde500cab5e6b21882 100644 (file)
@@ -75,8 +75,8 @@ public final class ModuleImpl extends AbstractDocumentedDataNodeContainer implem
         this.submodules = ImmutableSet.<Module> copyOf(builder.submodules);
         this.prefix = builder.getPrefix();
 
-        this.qnameModule = QNameModule.cachedReference(QNameModule.create(builder.getNamespace(),
-                builder.getRevision() == null ? null : new Date(builder.getRevision().getTime())));
+        this.qnameModule = QNameModule.create(builder.getNamespace(),
+                builder.getRevision() == null ? null : new Date(builder.getRevision().getTime())).intern();
         this.yangVersion = builder.getYangVersion();
         this.organization = builder.getOrganization();
         this.contact = builder.getContact();
index 0a3c7fe6ef196db780268b21a9783437fc5a82ff..1254e2d5f726a43301e8c83acb066ef840ea5b8a 100644 (file)
@@ -600,7 +600,7 @@ public final class YangParserListenerImpl extends YangParserBaseListener {
                     namespace = namespaces.get(revision);
                 }
 
-                final QNameModule mod = QNameModule.cachedReference(QNameModule.create(namespace, revision));
+                final QNameModule mod = QNameModule.create(namespace, revision).intern();
                 qname = QName.create(mod, name);
             }
         }
index 1f68421d2cf0e33583b76245074b87bb024a34a0..a6e5f0c0bd711b652915b168bb4529db37111cd6 100644 (file)
@@ -106,7 +106,7 @@ public class ModuleStatementSupport extends
             revisionDate = Optional.of(SimpleDateFormatUtil.DEFAULT_DATE_REV);
         }
 
-        QNameModule qNameModule = QNameModule.cachedReference(QNameModule.create(moduleNs.get(), revisionDate.orNull()));
+        QNameModule qNameModule = QNameModule.create(moduleNs.get(), revisionDate.orNull()).intern();
         ModuleIdentifier moduleIdentifier = new ModuleIdentifierImpl(stmt.getStatementArgument(),
                 Optional.<URI> absent(), revisionDate);
 
@@ -134,4 +134,4 @@ public class ModuleStatementSupport extends
         super.onFullDefinitionDeclared(stmt);
         SUBSTATEMENT_VALIDATOR.validate(stmt);
     }
-}
\ No newline at end of file
+}
index 62a5944f1cc574b2f8a9d9acc8783728dfc1194e..1d976c5aa08ca8b77c2ef996845625e111b80d42 100644 (file)
@@ -246,8 +246,8 @@ public final class Utils {
                 ctx.getRoot().rawStatementArgument(), value);
         final QNameModule resultQNameModule;
         if (qNameModule.getRevision() == null) {
-            resultQNameModule = QNameModule.cachedReference(
-                QNameModule.create(qNameModule.getNamespace(), SimpleDateFormatUtil.DEFAULT_DATE_REV));
+            resultQNameModule = QNameModule.create(qNameModule.getNamespace(), SimpleDateFormatUtil.DEFAULT_DATE_REV)
+                .intern();
         } else {
             resultQNameModule = qNameModule;
         }
@@ -288,8 +288,7 @@ public final class Utils {
             return qNameModule;
         }
 
-        return QNameModule.cachedReference(
-            QNameModule.create(qNameModule.getNamespace(), SimpleDateFormatUtil.DEFAULT_DATE_REV));
+        return QNameModule.create(qNameModule.getNamespace(), SimpleDateFormatUtil.DEFAULT_DATE_REV).intern();
     }
 
     @Nullable
index 20103be894be302a7201bec1ea9d3380645e0351..3ba7ff4f980b49f4713307c2deeb0fcac75a1baf 100644 (file)
@@ -24,8 +24,7 @@ public final class ModuleEffectiveStatementImpl extends AbstractEffectiveModule<
 
         final QNameModule module = ctx.getFromNamespace(ModuleCtxToModuleQName.class, ctx);
         if (module.getRevision() == null) {
-            qNameModule = QNameModule.cachedReference(
-                QNameModule.create(module.getNamespace(), SimpleDateFormatUtil.DEFAULT_DATE_REV));
+            qNameModule = QNameModule.create(module.getNamespace(), SimpleDateFormatUtil.DEFAULT_DATE_REV).intern();
         } else {
             qNameModule = module;
         }
index 7c185e7ac60045b6fc447e2f5603a83e9ba0de35..dc3af809c769606b4cf19caeffabb80a734243a2 100644 (file)
@@ -30,9 +30,9 @@ public final class SubmoduleEffectiveStatementImpl extends AbstractEffectiveModu
                 belongsToModuleName);
         RevisionEffectiveStatementImpl submoduleRevision = firstEffective(RevisionEffectiveStatementImpl.class);
 
-        this.qNameModule = QNameModule.cachedReference(submoduleRevision == null ?
+        this.qNameModule = (submoduleRevision == null ?
                 QNameModule.create(belongsToModuleQName.getNamespace(), SimpleDateFormatUtil.DEFAULT_DATE_REV) :
-                    QNameModule.create(belongsToModuleQName.getNamespace(), submoduleRevision.argument()));
+                    QNameModule.create(belongsToModuleQName.getNamespace(), submoduleRevision.argument())).intern();
     }
 
     @Override