Return Ipv{4,6}AddressNoZone when formatting from binary

Ever since MDSAL-440 was implemented, there is no point in our
utilities returning Ipv4Address objects as they never contain a zone
and therefore are equivalent to Ipv4AddressNoZone.

Ipv4AddressNoZone has the additional benefit that the lack of a zone
is contained in the type definition -- hence it can be propagated
through Java interfaces in a type-safe manner. Futhermore, given
the relationship between the two classes, the non-presence of zone
can be ascertained via a simple instanceof check rather than
searching the string.

JIRA: MDSAL-511
Change-Id: I7e0ed85bad7a63af9e9a6f8838609c0e53776ab0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/model/ietf/ietf-type-util/src/main/java/org/opendaylight/mdsal/model/ietf/util/AbstractIetfInetUtil.java b/model/ietf/ietf-type-util/src/main/java/org/opendaylight/mdsal/model/ietf/util/AbstractIetfInetUtil.java
index 24e35c3d8aa5960a89edfb049f2b241630c49323..124318ae5957437e0f00539634c86766dba866a5 100644 (file)
--- a/model/ietf/ietf-type-util/src/main/java/org/opendaylight/mdsal/model/ietf/util/AbstractIetfInetUtil.java
+++ b/model/ietf/ietf-type-util/src/main/java/org/opendaylight/mdsal/model/ietf/util/AbstractIetfInetUtil.java
@@ -31,33 +31,28 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
     private static final int INET4_LENGTH = 4;
     private static final int INET6_LENGTH = 16;
 
     private static final int INET4_LENGTH = 4;
     private static final int INET6_LENGTH = 16;
 

-    private final StringValueObjectFactory<A4> address4Factory;

     private final StringValueObjectFactory<A4NZ> address4NoZoneFactory;
     private final StringValueObjectFactory<P4> prefix4Factory;
     private final StringValueObjectFactory<A4NZ> address4NoZoneFactory;
     private final StringValueObjectFactory<P4> prefix4Factory;

-    private final StringValueObjectFactory<A6> address6Factory;

     private final StringValueObjectFactory<A6NZ> address6NoZoneFactory;
     private final StringValueObjectFactory<P6> prefix6Factory;
     private final Class<A4NZ> addr4nzClass;
     private final Class<A6NZ> addr6nzClass;
 
     private final StringValueObjectFactory<A6NZ> address6NoZoneFactory;
     private final StringValueObjectFactory<P6> prefix6Factory;
     private final Class<A4NZ> addr4nzClass;
     private final Class<A6NZ> addr6nzClass;
 

-    protected AbstractIetfInetUtil(final Class<A4> addr4Class, final Class<A4NZ> addr4nzClass,
-            final Class<P4> prefix4Class, final Class<A6> addr6Class, final Class<A6NZ> addr6nzClass,
-            final Class<P6> prefix6Class) {
+    protected AbstractIetfInetUtil(final Class<A4NZ> addr4nzClass, final Class<P4> prefix4Class,
+            final Class<A6NZ> addr6nzClass, final Class<P6> prefix6Class) {

         this.addr4nzClass = requireNonNull(addr4nzClass);
         this.addr6nzClass = requireNonNull(addr6nzClass);
         this.addr4nzClass = requireNonNull(addr4nzClass);
         this.addr6nzClass = requireNonNull(addr6nzClass);

-        this.address4Factory = StringValueObjectFactory.create(addr4Class, "0.0.0.0");

         this.address4NoZoneFactory = StringValueObjectFactory.create(addr4nzClass, "0.0.0.0");
         this.prefix4Factory = StringValueObjectFactory.create(prefix4Class, "0.0.0.0/0");
         this.address4NoZoneFactory = StringValueObjectFactory.create(addr4nzClass, "0.0.0.0");
         this.prefix4Factory = StringValueObjectFactory.create(prefix4Class, "0.0.0.0/0");

-        this.address6Factory = StringValueObjectFactory.create(addr6Class, "::0");

         this.address6NoZoneFactory = StringValueObjectFactory.create(addr6nzClass, "::0");
         this.prefix6Factory = StringValueObjectFactory.create(prefix6Class, "::0/0");
     }
 
         this.address6NoZoneFactory = StringValueObjectFactory.create(addr6nzClass, "::0");
         this.prefix6Factory = StringValueObjectFactory.create(prefix6Class, "::0/0");
     }
 

-    protected abstract @NonNull A ipv4Address(@NonNull A4 addr);
+    protected abstract @NonNull A ipv4Address(@NonNull A4NZ addr);

 
     protected abstract @NonNull ANZ ipv4AddressNoZone(@NonNull A4NZ addr);
 
 
     protected abstract @NonNull ANZ ipv4AddressNoZone(@NonNull A4NZ addr);
 

-    protected abstract @NonNull A ipv6Address(@NonNull A6 addr);
+    protected abstract @NonNull A ipv6Address(@NonNull A6NZ addr);

 
     protected abstract @NonNull ANZ ipv6AddressNoZone(@NonNull A6NZ addr);
 
 
     protected abstract @NonNull ANZ ipv6AddressNoZone(@NonNull A6NZ addr);
 


@@ -122,9 +117,9 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
     public final @NonNull ANZ ipAddressNoZoneFor(final byte @NonNull[] bytes) {
         switch (bytes.length) {
             case INET4_LENGTH:
     public final @NonNull ANZ ipAddressNoZoneFor(final byte @NonNull[] bytes) {
         switch (bytes.length) {
             case INET4_LENGTH:

-                return ipv4AddressNoZone(ipv4AddressNoZoneFor(bytes));
+                return ipv4AddressNoZone(ipv4AddressFor(bytes));

             case INET6_LENGTH:
             case INET6_LENGTH:

-                return ipv6AddressNoZone(ipv6AddressNoZoneFor(bytes));
+                return ipv6AddressNoZone(ipv6AddressFor(bytes));

             default:
                 throw new IllegalArgumentException("Invalid array length " + bytes.length);
         }
             default:
                 throw new IllegalArgumentException("Invalid array length " + bytes.length);
         }


@@ -133,9 +128,9 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
     public final @NonNull ANZ ipAddressNoZoneFor(final @NonNull InetAddress addr) {
         requireNonNull(addr, "Address must not be null");
         if (addr instanceof Inet4Address) {
     public final @NonNull ANZ ipAddressNoZoneFor(final @NonNull InetAddress addr) {
         requireNonNull(addr, "Address must not be null");
         if (addr instanceof Inet4Address) {

-            return ipv4AddressNoZone(ipv4AddressNoZoneFor(addr));
+            return ipv4AddressNoZone(ipv4AddressFor(addr));

         } else if (addr instanceof Inet6Address) {
         } else if (addr instanceof Inet6Address) {

-            return ipv6AddressNoZone(ipv6AddressNoZoneFor(addr));
+            return ipv6AddressNoZone(ipv6AddressFor(addr));

         } else {
             throw new IllegalArgumentException("Unhandled address " + addr);
         }
         } else {
             throw new IllegalArgumentException("Unhandled address " + addr);
         }


@@ -229,37 +224,37 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
     }
 
     /**
     }
 
     /**

-     * Create an Ipv4Address by interpreting input bytes as an IPv4 address.
+     * Create an Ipv4AddressNoZone by interpreting input bytes as an IPv4 address.

      *
      * @param bytes 4-byte array
      *
      * @param bytes 4-byte array

-     * @return An Ipv4Address object
+     * @return An Ipv4AddressNoZone object

      * @throws IllegalArgumentException if bytes has length different from 4
      * @throws NullPointerException if bytes is null
      */
      * @throws IllegalArgumentException if bytes has length different from 4
      * @throws NullPointerException if bytes is null
      */

-    public final @NonNull A4 ipv4AddressFor(final byte @NonNull[] bytes) {
-        return address4Factory.newInstance(addressStringV4(bytes));
+    public final @NonNull A4NZ ipv4AddressFor(final byte @NonNull[] bytes) {
+        return address4NoZoneFactory.newInstance(addressStringV4(bytes));

     }
 
     /**
     }
 
     /**

-     * Create an Ipv4Address by interpreting an {@link Inet4Address}.
+     * Create an Ipv4AddressNoZone by interpreting an {@link Inet4Address}.

      *
      * @param addr An {@link Inet4Address}
      *
      * @param addr An {@link Inet4Address}

-     * @return An Ipv4Address object
+     * @return An Ipv4AddressNoZone object

      * @throws IllegalArgumentException if addr is not an {@link Inet4Address}
      * @throws NullPointerException if addr is null
      */
      * @throws IllegalArgumentException if addr is not an {@link Inet4Address}
      * @throws NullPointerException if addr is null
      */

-    public final @NonNull A4 ipv4AddressFor(final @NonNull InetAddress addr) {
-        return address4Factory.newInstance(addressStringV4(addr));
+    public final @NonNull A4NZ ipv4AddressFor(final @NonNull InetAddress addr) {
+        return address4NoZoneFactory.newInstance(addressStringV4(addr));

     }
 
     /**
     }
 
     /**

-     * Create an Ipv4Address by interpreting input 32 bits as an IPv4 address in big-endian format.
+     * Create an Ipv4AddressNoZone by interpreting input 32 bits as an IPv4 address in big-endian format.

      *
      * @param bits 32 bits, big endian
      *
      * @param bits 32 bits, big endian

-     * @return An Ipv4Address object
+     * @return An Ipv4AddressNoZone object

      */
      */

-    public final @NonNull A4 ipv4AddressFor(final int bits) {
-        return address4Factory.newInstance(Ipv4Utils.addressString(bits));
+    public final @NonNull A4NZ ipv4AddressFor(final int bits) {
+        return address4NoZoneFactory.newInstance(Ipv4Utils.addressString(bits));

     }
 
     /**
     }
 
     /**


@@ -269,9 +264,11 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
      * @return An Ipv4AddressNoZone object
      * @throws IllegalArgumentException if bytes has length different from 4
      * @throws NullPointerException if bytes is null
      * @return An Ipv4AddressNoZone object
      * @throws IllegalArgumentException if bytes has length different from 4
      * @throws NullPointerException if bytes is null

+     * @deprecated Use {@link #ipv4AddressFor(byte[])} instead.

      */
      */

+    @Deprecated(forRemoval = true)

     public final @NonNull A4NZ ipv4AddressNoZoneFor(final byte @NonNull[] bytes) {
     public final @NonNull A4NZ ipv4AddressNoZoneFor(final byte @NonNull[] bytes) {

-        return address4NoZoneFactory.newInstance(addressStringV4(bytes));
+        return ipv4AddressFor(bytes);

     }
 
     /**
     }
 
     /**


@@ -281,9 +278,11 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
      * @return An Ipv4AddressNoZone object
      * @throws IllegalArgumentException if addr is not an {@link Inet4Address}
      * @throws NullPointerException if addr is null
      * @return An Ipv4AddressNoZone object
      * @throws IllegalArgumentException if addr is not an {@link Inet4Address}
      * @throws NullPointerException if addr is null

+     * @deprecated Use {@link #ipv4AddressFor(InetAddress)} instead.

      */
      */

+    @Deprecated(forRemoval = true)

     public final @NonNull A4NZ ipv4AddressNoZoneFor(final @NonNull InetAddress addr) {
     public final @NonNull A4NZ ipv4AddressNoZoneFor(final @NonNull InetAddress addr) {

-        return address4NoZoneFactory.newInstance(addressStringV4(addr));
+        return ipv4AddressFor(addr);

     }
 
     /**
     }
 
     /**


@@ -291,9 +290,11 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
      *
      * @param bits 32 bits, big endian
      * @return An Ipv4AddressNoZone object
      *
      * @param bits 32 bits, big endian
      * @return An Ipv4AddressNoZone object

+     * @deprecated Use {@link #ipv4AddressFor(int)} instead.

      */
      */

+    @Deprecated(forRemoval = true)

     public final @NonNull A4NZ ipv4AddressNoZoneFor(final int bits) {
     public final @NonNull A4NZ ipv4AddressNoZoneFor(final int bits) {

-        return address4NoZoneFactory.newInstance(Ipv4Utils.addressString(bits));
+        return ipv4AddressFor(bits);

     }
 
     /**
     }
 
     /**


@@ -309,12 +310,13 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
                 : address4NoZoneFactory.newInstance(stripZone(ipv4AddressString(addr)));
     }
 
                 : address4NoZoneFactory.newInstance(stripZone(ipv4AddressString(addr)));
     }
 

-    public final @NonNull A4 ipv4AddressFrom(final @NonNull P4 prefix) {
-        return prefixToAddress(address4Factory, ipv4PrefixString(prefix));
+    public final @NonNull A4NZ ipv4AddressFrom(final @NonNull P4 prefix) {
+        return prefixToAddress(address4NoZoneFactory, ipv4PrefixString(prefix));

     }
 
     }
 

+    @Deprecated(forRemoval = true)

     public final @NonNull A4NZ ipv4AddressNoZoneFrom(final @NonNull P4 prefix) {
     public final @NonNull A4NZ ipv4AddressNoZoneFrom(final @NonNull P4 prefix) {

-        return prefixToAddress(address4NoZoneFactory, ipv4PrefixString(prefix));
+        return ipv4AddressFrom(prefix);

     }
 
     public final byte @NonNull[] ipv4AddressBytes(final @NonNull A4 addr) {
     }
 
     public final byte @NonNull[] ipv4AddressBytes(final @NonNull A4 addr) {


@@ -448,12 +450,13 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
         return prefix4Factory.newInstance(addr + '/' + mask);
     }
 
         return prefix4Factory.newInstance(addr + '/' + mask);
     }
 

-    public final @NonNull Entry<A4, Integer> splitIpv4Prefix(final @NonNull P4 prefix) {
-        return splitPrefix(address4Factory, ipv4PrefixString(prefix));
+    public final @NonNull Entry<A4NZ, Integer> splitIpv4Prefix(final @NonNull P4 prefix) {
+        return splitPrefix(address4NoZoneFactory, ipv4PrefixString(prefix));

     }
 
     }
 

+    @Deprecated(forRemoval = true)

     public final @NonNull Entry<A4NZ, Integer> splitIpv4PrefixNoZone(final @NonNull P4 prefix) {
     public final @NonNull Entry<A4NZ, Integer> splitIpv4PrefixNoZone(final @NonNull P4 prefix) {

-        return splitPrefix(address4NoZoneFactory, ipv4PrefixString(prefix));
+        return splitIpv4Prefix(prefix);

     }
 
     public final byte @NonNull[] ipv4PrefixToBytes(final @NonNull P4 prefix) {
     }
 
     public final byte @NonNull[] ipv4PrefixToBytes(final @NonNull P4 prefix) {


@@ -474,8 +477,8 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
      * @throws IllegalArgumentException if bytes has length different from 16
      * @throws NullPointerException if bytes is null
      */
      * @throws IllegalArgumentException if bytes has length different from 16
      * @throws NullPointerException if bytes is null
      */

-    public final @NonNull A6 ipv6AddressFor(final byte @NonNull[] bytes) {
-        return address6Factory.newInstance(addressStringV6(bytes));
+    public final @NonNull A6NZ ipv6AddressFor(final byte @NonNull[] bytes) {
+        return address6NoZoneFactory.newInstance(addressStringV6(bytes));

     }
 
     /**
     }
 
     /**


@@ -486,8 +489,8 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
      * @throws IllegalArgumentException if addr is not an {@link Inet6Address}
      * @throws NullPointerException if addr is null
      */
      * @throws IllegalArgumentException if addr is not an {@link Inet6Address}
      * @throws NullPointerException if addr is null
      */

-    public final @NonNull A6 ipv6AddressFor(final @NonNull InetAddress addr) {
-        return address6Factory.newInstance(addressStringV6(addr));
+    public final @NonNull A6NZ ipv6AddressFor(final @NonNull InetAddress addr) {
+        return address6NoZoneFactory.newInstance(addressStringV6(addr));

     }
 
     /**
     }
 
     /**


@@ -497,9 +500,11 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
      * @return An Ipv6Address object
      * @throws IllegalArgumentException if bytes has length different from 16
      * @throws NullPointerException if bytes is null
      * @return An Ipv6Address object
      * @throws IllegalArgumentException if bytes has length different from 16
      * @throws NullPointerException if bytes is null

+     * @deprecated Use {@link #ipv6AddressFor(byte[])} instead.

      */
      */

+    @Deprecated(forRemoval = true)

     public final @NonNull A6NZ ipv6AddressNoZoneFor(final byte @NonNull[] bytes) {
     public final @NonNull A6NZ ipv6AddressNoZoneFor(final byte @NonNull[] bytes) {

-        return address6NoZoneFactory.newInstance(addressStringV6(bytes));
+        return ipv6AddressFor(bytes);

     }
 
     /**
     }
 
     /**


@@ -509,9 +514,11 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
      * @return An Ipv6AddressNoZone object
      * @throws IllegalArgumentException if addr is not an {@link Inet6Address}
      * @throws NullPointerException if addr is null
      * @return An Ipv6AddressNoZone object
      * @throws IllegalArgumentException if addr is not an {@link Inet6Address}
      * @throws NullPointerException if addr is null

+     * @deprecated Use {@link #ipv6AddressFor(InetAddress)} instead.

      */
      */

+    @Deprecated(forRemoval = true)

     public final @NonNull A6NZ ipv6AddressNoZoneFor(final @NonNull InetAddress addr) {
     public final @NonNull A6NZ ipv6AddressNoZoneFor(final @NonNull InetAddress addr) {

-        return address6NoZoneFactory.newInstance(addressStringV6(addr));
+        return ipv6AddressFor(addr);

     }
 
     /**
     }
 
     /**


@@ -527,12 +534,13 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
                 : address6NoZoneFactory.newInstance(stripZone(ipv6AddressString(addr)));
     }
 
                 : address6NoZoneFactory.newInstance(stripZone(ipv6AddressString(addr)));
     }
 

-    public final @NonNull A6 ipv6AddressFrom(final @NonNull P6 prefix) {
-        return prefixToAddress(address6Factory, ipv6PrefixString(prefix));
+    public final @NonNull A6NZ ipv6AddressFrom(final @NonNull P6 prefix) {
+        return prefixToAddress(address6NoZoneFactory, ipv6PrefixString(prefix));

     }
 
     }
 

+    @Deprecated(forRemoval = true)

     public final @NonNull A6NZ ipv6AddressNoZoneFrom(final @NonNull P6 prefix) {
     public final @NonNull A6NZ ipv6AddressNoZoneFrom(final @NonNull P6 prefix) {

-        return prefixToAddress(address6NoZoneFactory, ipv6PrefixString(prefix));
+        return ipv6AddressFrom(prefix);

     }
 
     public final byte @NonNull[] ipv6AddressBytes(final @NonNull A6 addr) {
     }
 
     public final byte @NonNull[] ipv6AddressBytes(final @NonNull A6 addr) {


@@ -652,12 +660,13 @@ public abstract class AbstractIetfInetUtil<A4, A4NZ extends A4, P4, A6, A6NZ ext
         return prefix6Factory.newInstance(addr + '/' + mask);
     }
 
         return prefix6Factory.newInstance(addr + '/' + mask);
     }
 

-    public final @NonNull Entry<A6, Integer> splitIpv6Prefix(final @NonNull P6 prefix) {
-        return splitPrefix(address6Factory, ipv6PrefixString(prefix));
+    public final @NonNull Entry<A6NZ, Integer> splitIpv6Prefix(final @NonNull P6 prefix) {
+        return splitPrefix(address6NoZoneFactory, ipv6PrefixString(prefix));

     }
 
     }
 

+    @Deprecated(forRemoval = true)

     public final @NonNull Entry<A6NZ, Integer> splitIpv6PrefixNoZone(final @NonNull P6 prefix) {
     public final @NonNull Entry<A6NZ, Integer> splitIpv6PrefixNoZone(final @NonNull P6 prefix) {

-        return splitPrefix(address6NoZoneFactory, ipv6PrefixString(prefix));
+        return splitIpv6Prefix(prefix);

     }
 
     private static <T> @NonNull T prefixToAddress(final StringValueObjectFactory<T> factory, final String str) {
     }
 
     private static <T> @NonNull T prefixToAddress(final StringValueObjectFactory<T> factory, final String str) {

diff --git a/model/ietf/ietf-type-util/src/test/java/org/opendaylight/mdsal/model/ietf/util/IpUtil.java b/model/ietf/ietf-type-util/src/test/java/org/opendaylight/mdsal/model/ietf/util/IpUtil.java
index bf4756441c089e5d2aee8ccb3222676ede118089..e28e16fd4d480ea58edd5d84cedb629c45c01771 100644 (file)
--- a/model/ietf/ietf-type-util/src/test/java/org/opendaylight/mdsal/model/ietf/util/IpUtil.java
+++ b/model/ietf/ietf-type-util/src/test/java/org/opendaylight/mdsal/model/ietf/util/IpUtil.java
@@ -18,7 +18,7 @@ final class IpUtil extends AbstractIetfInetUtil<IpClass, IpClass, IpClass, IpCla
     private static final Pattern IP_V4_PATTERN = Pattern.compile(IP_V4_REGEX);
 
     IpUtil() {
     private static final Pattern IP_V4_PATTERN = Pattern.compile(IP_V4_REGEX);
 
     IpUtil() {

-        super(IpClass.class, IpClass.class, IpClass.class, IpClass.class, IpClass.class, IpClass.class);
+        super(IpClass.class, IpClass.class, IpClass.class, IpClass.class);

     }
 
     @Override
     }
 
     @Override

diff --git a/model/ietf/rfc6991-ietf-inet-types/src/main/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtil.java b/model/ietf/rfc6991-ietf-inet-types/src/main/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtil.java
index 1aa9c7b3f9149c93c829ab15076330ad880dcc04..1a43a761f894bcc1b7dba478a611b5a68a489f94 100644 (file)
--- a/model/ietf/rfc6991-ietf-inet-types/src/main/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtil.java
+++ b/model/ietf/rfc6991-ietf-inet-types/src/main/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtil.java
@@ -19,12 +19,11 @@ public final class IetfInetUtil extends AbstractIetfInetUtil<Ipv4Address, Ipv4Ad
     public static final IetfInetUtil INSTANCE = new IetfInetUtil();
 
     private IetfInetUtil() {
     public static final IetfInetUtil INSTANCE = new IetfInetUtil();
 
     private IetfInetUtil() {

-        super(Ipv4Address.class, Ipv4AddressNoZone.class, Ipv4Prefix.class, Ipv6Address.class, Ipv6AddressNoZone.class,
-            Ipv6Prefix.class);
+        super(Ipv4AddressNoZone.class, Ipv4Prefix.class, Ipv6AddressNoZone.class, Ipv6Prefix.class);

     }
 
     @Override
     }
 
     @Override

-    protected IpAddress ipv4Address(final Ipv4Address addr) {
+    protected IpAddress ipv4Address(final Ipv4AddressNoZone addr) {

         return new IpAddress(addr);
     }
 
         return new IpAddress(addr);
     }
 


@@ -39,7 +38,7 @@ public final class IetfInetUtil extends AbstractIetfInetUtil<Ipv4Address, Ipv4Ad
     }
 
     @Override
     }
 
     @Override

-    protected IpAddress ipv6Address(final Ipv6Address addr) {
+    protected IpAddress ipv6Address(final Ipv6AddressNoZone addr) {

         return new IpAddress(addr);
     }
 
         return new IpAddress(addr);
     }
 

diff --git a/model/ietf/rfc6991-ietf-inet-types/src/test/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtilTest.java b/model/ietf/rfc6991-ietf-inet-types/src/test/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtilTest.java
index 141c5fa509ed898507e1492e79ca51c876ec412e..fd48a656ed15d87f70789565959d9cf101385bd1 100644 (file)
--- a/model/ietf/rfc6991-ietf-inet-types/src/test/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtilTest.java
+++ b/model/ietf/rfc6991-ietf-inet-types/src/test/java/org/opendaylight/yang/gen/v1/urn/ietf/params/xml/ns/yang/ietf/inet/types/rev130715/IetfInetUtilTest.java
@@ -29,7 +29,7 @@ public class IetfInetUtilTest {
 
     @Test
     public void testIpv4Address() {
 
     @Test
     public void testIpv4Address() {

-        final Ipv4Address ipv4Address = new Ipv4Address("192.168.1.1");
+        final Ipv4AddressNoZone ipv4Address = new Ipv4AddressNoZone("192.168.1.1");

         final Ipv4Prefix ipv4Prefix = new Ipv4Prefix("192.0.2.1/24");
         final IpAddress ipAddress = IetfInetUtil.INSTANCE.ipv4Address(ipv4Address);
         final String ipv4AddressString = IetfInetUtil.INSTANCE.ipv4AddressString(ipv4Address);
         final Ipv4Prefix ipv4Prefix = new Ipv4Prefix("192.0.2.1/24");
         final IpAddress ipAddress = IetfInetUtil.INSTANCE.ipv4Address(ipv4Address);
         final String ipv4AddressString = IetfInetUtil.INSTANCE.ipv4AddressString(ipv4Address);


@@ -45,7 +45,7 @@ public class IetfInetUtilTest {
 
     @Test
     public void testIpv6Address() {
 
     @Test
     public void testIpv6Address() {

-        final Ipv6Address ipv6Address = new Ipv6Address("ABCD:EF01:2345:6789:ABCD:EF01:2345:6789");
+        final Ipv6AddressNoZone ipv6Address = new Ipv6AddressNoZone("ABCD:EF01:2345:6789:ABCD:EF01:2345:6789");

         final Ipv6Prefix ipv6Prefix = new Ipv6Prefix("ff00::/8");
         final IpAddress ipAddress = IetfInetUtil.INSTANCE.ipv6Address(ipv6Address);
         final String ipv6AddressString = IetfInetUtil.INSTANCE.ipv6AddressString(ipv6Address);
         final Ipv6Prefix ipv6Prefix = new Ipv6Prefix("ff00::/8");
         final IpAddress ipAddress = IetfInetUtil.INSTANCE.ipv6Address(ipv6Address);
         final String ipv6AddressString = IetfInetUtil.INSTANCE.ipv6AddressString(ipv6Address);