https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/MDSAL-704 mdsal <p>mdsal-binding-dom-codec is using yangtools.concepts.IllegalArgumentCodec, which is going away. The use here is not conforming to the @NonNull requirement anyway.</p> <p>Add a dom-codec-local replacement with exactly the semantics we need, probably with a checked exception being thrown, and use that instead of IllegalArgumentCodec.</p> MDSAL-704

Do not use IllegalArgumentCodec

Task Medium Resolved Done Robert Varga Robert Varga pt Thu, 28 Oct 2021 10:40:25 +0000 Fri, 10 Jun 2022 12:49:11 +0000 Fri, 10 Jun 2022 12:49:11 +0000 10.0.0 Binding runtime 0 2 <p>Do you wish to have a new codec in org.opendaylight.mdsal.binding.dom.codec.api package together with another codec<font color="#a9b7c6"></font></p> <p>or should it have its own dom-codec-local package/bundle?</p> <p>Purely local thing would be best, no sense in proliferating single-use things <img class="emoticon" src="https://jira.opendaylight.org/images/icons/emoticons/smile.png" height="16" width="16" align="absmiddle" alt="" border="0"/></p> <p>So going through the analysis, we can do quite a bit of untangling, as it seem all codecs are null-hostile. The one outlier is InstanceIdentifier, which produces nulls when a YangInstanceIdentifier is not representable. Let's juggle the class hierarchy first, and let's see what possibilities fall out of this.</p> <p>We have a mdsal-binding-dom-codec-api construct in BindingIdentityCodec, which pretty much follows what we want to do. There is also a counterpart in BindingInstanceIdentifierCodec, hence we should be able to generalize these constructs. Since this is an API change, move implementation to 10.0.0.</p> Development Epic Link YANGTOOLS-1332 Rank 0|i040hr: