On Mon, 15 Oct 2018 10:37:26 +0200 Robert Varga <[email protected]> wrote: > On 15/10/2018 10:32, Stephen Kitt wrote: > > On Sun, 14 Oct 2018 20:45:31 +0200 > > Robert Varga <[email protected]> wrote: > >> The question is what are we going to do about this? > >> > >> One option is to implement > >> https://jira.opendaylight.org/browse/MDSAL-18 for LazyDataObjects > >> where the list is not restricted by min-elements > 0. > >> > >> Another is to have a utility method which turns nulls to empty > >> lists. > > Do you know how much effort the first option would take? (Going by > > the comments on MDSAL-18, I get the impression you’ve already > > started looking into it!) It would seem to be better in the long > > run... > > The codec part is rather trivial, but the problem is DTOs -- it > requires changing equality contracts and it affects all DTO use, not > just where data store is involved. > > While if felt feasible yesterday, I cannot quite wrap my mind around > impacts and what can break and where. I don't thing this really is an > option before mdsal-4.0.0.
OK, so I’ve run a null-pointer analysis on NetVirt and am fixing the affected call-sites using the equivalent of Java 9’s Objects.requireNonNullElse(). Regards, Stephen
pgpnyQZIMSXbM.pgp
Description: OpenPGP digital signature
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
