From: Lukas Sedlak <lsedlak@cisco.com>
Date: Thu, 4 Sep 2014 14:38:23 +0000 (+0200)
Subject: Bug 576 Fixes of critical issues for YT components
X-Git-Tag: release/beryllium~339^2
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=commitdiff_plain;h=8139b0f1aa2f50c3cb7330776425deda3a0d0734;p=mdsal.git

Bug 576 Fixes of critical issues for YT components

This is fix for https://bugs.opendaylight.org/show_bug.cgi?id=576
Added Nonnull annotations and preconditions checks for annonymous apply methods.
Fixed never reachable if statement in writeValue(final XMLStreamWriter writer, final @Nonnull Node<?> data, final SchemaNode schema) in XmlStreamUtils
since instanceof is doing already null check.
Removed empty if statement in NormalizedNodeContainerModificationStrategy in verifyStructure method.
Removed dead code from in PerRepoTestRunner runnersFromFeatures method.

Change-Id: Icec2a30149a17d8b76497d95ac0054297369e985
Signed-off-by: Lukas Sedlak <lsedlak@cisco.com>
---

diff --git a/code-generator/binding-generator-impl/src/main/java/org/opendaylight/yangtools/sal/binding/generator/impl/LazyGeneratedCodecRegistry.java b/code-generator/binding-generator-impl/src/main/java/org/opendaylight/yangtools/sal/binding/generator/impl/LazyGeneratedCodecRegistry.java
index 65e80d0584..f600d8fbdf 100644
--- a/code-generator/binding-generator-impl/src/main/java/org/opendaylight/yangtools/sal/binding/generator/impl/LazyGeneratedCodecRegistry.java
+++ b/code-generator/binding-generator-impl/src/main/java/org/opendaylight/yangtools/sal/binding/generator/impl/LazyGeneratedCodecRegistry.java
@@ -679,8 +679,7 @@ class LazyGeneratedCodecRegistry implements CodecRegistry, SchemaContextListener
             if (implementation == null) {
                 implementation = tryToLoadImplementationImpl(inputType);
             }
-
-            return null;
+            return implementation;
         }
 
         private T tryToLoadImplementationImpl(final Class<? extends DataContainer> inputType) {