Validate namespace sooner

Pass the read string directly to XMLNamespace.of() before we read the
revision.

Change-Id: I67f2a7f258d340bba107b0233501061dd8af352b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/common/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QNameModule.java b/common/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QNameModule.java
index 0e0ffeb6c1873ee8fc58107d68c8811cb7d6682b..0441105afc9390ff0e8b295816632217f500bc5d 100644 (file)
--- a/common/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QNameModule.java
+++ b/common/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QNameModule.java
@@ -98,9 +98,9 @@ public final class QNameModule implements Comparable<QNameModule>, Immutable, Se
      * @throws IOException if I/O error occurs
      */
     public static @NonNull QNameModule readFrom(final DataInput in) throws IOException {
-        final String namespace = in.readUTF();
-        final String revision = in.readUTF();
-        return new QNameModule(XMLNamespace.of(namespace), revision.isEmpty() ? null : Revision.of(revision));
+        final var namespace = XMLNamespace.of(in.readUTF());
+        final var revStr = in.readUTF();
+        return new QNameModule(namespace, revStr.isEmpty() ? null : Revision.of(revStr));
     }
 
     /**