Re: [controller-dev] Checkstyle failing when using Union typedefs
Robert Varga
Actually RFC6020 defines the evaluation order, so this should get eliminated as an enhancement. Not sure if we have one -- can you check/file one?
Thanks,
Robert
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
Thanks,
Robert
On October 24, 2014 9:34:10 AM CEST, "Tony Tkacik -X (ttkacik - Pantheon Technologies SRO at Cisco)" <ttkacik@...> wrote:
The generated classes for Unions with name *Builder are targeted for developer
to be modified manually and fill out to implement missing code (selection between
union subtypes, since if you have union of string and int) some preference code
/logic needs to be handcodec.
That failure currently is due to missing imports, but still idea is that you as developer
Of model will modify created class.
Tony
-----Original Message-----
From: controller-dev-bounces@... [mailto:controller-dev-bounces@...] On Behalf Of Sharon Aicler (saichler)
Sent: Friday, October 24, 2014 12:26 AM
To: controller-dev@...
Cc: integration-dev@...
Subject: [controller-dev] Checkstyle failing when using Union typedefs
Hi All,
FYI,
Whenever i use union types in my .yang files i get the following error:
INF/services/org.opendaylight.yangtools.yang.binding.YangModelBindingProvider]
[INFO]
[INFO] --- maven-checkstyle-plugin:2.12:check (default) @ sal-persisted-dom --- [INFO] Starting audit...
/root/ODL/controller/opendaylight/md-sal/sal-persisted-dom/src/main/java/org/opendaylight/yang/gen/v1/http/netconfcentral/org/ns/persistedtest/rev141020/TypedefUnionBuilder.java:2:1: Redundant import from the same package - org.opendaylight.yang.gen.v1.http.netconfcentral.org.ns.persistedtest.rev141020.TypedefUnion.
Audit done.
I always have to go and manually remove the import...
controller-dev mailing list
controller-dev@...
https://lists.opendaylight.org/mailman/listinfo/controller-dev
controller-dev mailing list
controller-dev@...
https://lists.opendaylight.org/mailman/listinfo/controller-dev
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.