Details
-
Task
-
Status: Resolved
-
Medium
-
Resolution: Done
-
None
Description
mdsal-binding-dom-codec is using yangtools.concepts.IllegalArgumentCodec, which is going away. The use here is not conforming to the @NonNull requirement anyway.
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.
Attachments
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 98374,5 | Eliminate use of the IllegalArgumentCodec | master | mdsal | Status: MERGED | +2 | +1 |
| 100530,1 | Disconnect IdentifiableItemCodec from IllegalArgumentCodec | master | mdsal | Status: MERGED | +2 | +1 |
| 100532,1 | Split STATIC_CODECS into per-type caches | master | mdsal | Status: MERGED | +2 | +1 |
| 100533,1 | Promote SchemaUnawareCodec to a top-level construct | master | mdsal | Status: MERGED | +2 | +1 |
| 100534,1 | EncapsulatedValueCodec does not use TypeDefinition | master | mdsal | Status: MERGED | +2 | +1 |
| 100535,2 | Disconnect EncapsulatedValueCodec from ReflectionBasedCodec | master | mdsal | Status: MERGED | +2 | +1 |
| 100536,1 | Eliminate ValueTypeCodec.encapsulatedValueCodecFor() | master | mdsal | Status: MERGED | +2 | +1 |
| 100537,1 | Disconnect BitsCodec from ReflectionBasedCodec | master | mdsal | Status: MERGED | +2 | +1 |
| 100538,1 | Remove ReflectionBaseCodec | master | mdsal | Status: MERGED | +2 | +1 |
| 100539,1 | Do not indirect UnionLoading through Callable | master | mdsal | Status: MERGED | +2 | +1 |
| 100540,1 | Switch SchemaUnawareCodec to an abstract class | master | mdsal | Status: MERGED | +2 | +1 |
| 100541,1 | SchemaUnawareCodec is AbstractIllegalArgumentCodec | master | mdsal | Status: MERGED | +2 | +1 |
| 100586,1 | Reparent UnionTypeCodec to AbstractIllegalArgumentCodec | master | mdsal | Status: ABANDONED | 0 | +1 |
| 100587,2 | Simplify CompositeValueCodec instantiation | master | mdsal | Status: MERGED | +2 | +1 |
| 100609,2 | Remove ValueTypeCodec | master | mdsal | Status: MERGED | +2 | +1 |
| 100610,3 | Specialize CompositeValueCodec | master | mdsal | Status: MERGED | +2 | +1 |