From 4cb759451927d919c5d6f9083ac6d72f31f97fc7 Mon Sep 17 00:00:00 2001
From: Martin Ciglan <mciglan@cisco.com>
Date: Mon, 8 Aug 2016 15:47:50 +0200
Subject: [PATCH] BUG-6316: Fix Bit and EnumPair's position/value types

Follow-up patch for Yangtools API change:
https://git.opendaylight.org/gerrit/#/c/42855/

This solves issue with distibution-check jenkins job, where
EnumPair.getValue() returns Integer instead of correct int type.

Change-Id: I459364d17747e0adf10ec021a0bd9cc3e316726d
Signed-off-by: Martin Ciglan <mciglan@cisco.com>
(cherry picked from commit 36a598c3b773cf67d0897354c2644fcadcd11f8f)
---
 .../sal/binding/model/api/Enumeration.java    |  2 +-
 .../model/api/type/builder/EnumBuilder.java   |  2 +-
 .../type/builder/EnumerationBuilderImpl.java  | 23 ++++---------------
 3 files changed, 7 insertions(+), 20 deletions(-)

diff --git a/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/Enumeration.java b/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/Enumeration.java
index 9ca7f43651..55f5c502af 100644
--- a/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/Enumeration.java
+++ b/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/Enumeration.java
@@ -68,6 +68,6 @@ public interface Enumeration extends GeneratedType {
          *
          * @return the value of the enumeration item.
          */
-        Integer getValue();
+        int getValue();
     }
 }
diff --git a/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/type/builder/EnumBuilder.java b/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/type/builder/EnumBuilder.java
index a3339b9276..c0a99b9845 100644
--- a/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/type/builder/EnumBuilder.java
+++ b/binding/mdsal-binding-generator-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/type/builder/EnumBuilder.java
@@ -39,7 +39,7 @@ public interface EnumBuilder extends Type {
      * @param name
      * @param value
      */
-    void addValue(final String name, final Integer value, final String description);
+    void addValue(final String name, final int value, final String description);
 
     /**
      *
diff --git a/binding/mdsal-binding-generator-util/src/main/java/org/opendaylight/yangtools/binding/generator/util/generated/type/builder/EnumerationBuilderImpl.java b/binding/mdsal-binding-generator-util/src/main/java/org/opendaylight/yangtools/binding/generator/util/generated/type/builder/EnumerationBuilderImpl.java
index 7a86e731ba..198d938e2d 100644
--- a/binding/mdsal-binding-generator-util/src/main/java/org/opendaylight/yangtools/binding/generator/util/generated/type/builder/EnumerationBuilderImpl.java
+++ b/binding/mdsal-binding-generator-util/src/main/java/org/opendaylight/yangtools/binding/generator/util/generated/type/builder/EnumerationBuilderImpl.java
@@ -78,7 +78,7 @@ public final class EnumerationBuilderImpl extends AbstractBaseType implements En
     }
 
     @Override
-    public void addValue(final String name, final Integer value, final String description) {
+    public void addValue(final String name, final int value, final String description) {
         final EnumPairImpl p = new EnumPairImpl(name, value, description);
         values = LazyCollections.lazyAdd(values, p);
         unmodifiableValues = Collections.unmodifiableList(values);
@@ -112,21 +112,9 @@ public final class EnumerationBuilderImpl extends AbstractBaseType implements En
     public void updateEnumPairsFromEnumTypeDef(final EnumTypeDefinition enumTypeDef) {
         final List<EnumPair> enums = enumTypeDef.getValues();
         if (enums != null) {
-            int listIndex = 0;
             for (final EnumPair enumPair : enums) {
                 if (enumPair != null) {
-                    final String enumPairName = enumPair.getName();
-                    Integer enumPairValue = enumPair.getValue();
-
-                    if (enumPairValue == null) {
-                        enumPairValue = listIndex;
-                    }
-                    else {
-                        listIndex = enumPairValue;
-                    }
-
-                    this.addValue(enumPairName, enumPairValue, enumPair.getDescription());
-                    listIndex++;
+                    this.addValue(enumPair.getName(), enumPair.getValue(), enumPair.getDescription());
                 }
             }
         }
@@ -137,10 +125,10 @@ public final class EnumerationBuilderImpl extends AbstractBaseType implements En
 
         private final String name;
         private final String mappedName;
-        private final Integer value;
+        private final int value;
         private final String description;
 
-        public EnumPairImpl(final String name, final Integer value, final String description) {
+        public EnumPairImpl(final String name, final int value, final String description) {
             super();
             this.name = name;
             this.mappedName = BindingMapping.getClassName(name);
@@ -159,7 +147,7 @@ public final class EnumerationBuilderImpl extends AbstractBaseType implements En
         }
 
         @Override
-        public Integer getValue() {
+        public int getValue() {
             return value;
         }
 
@@ -362,7 +350,6 @@ public final class EnumerationBuilderImpl extends AbstractBaseType implements En
 
         @Override
         public List<MethodSignature> getMethodDefinitions() {
-            // TODO Auto-generated method stub
             return Collections.emptyList();
         }
 
-- 
2.36.6