Code Review / yangtools.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 7fe0455) | patch
Cleanup AbstractValueNodeModificationStrategy 33/80033/1
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 30 Jan 2019 13:31:16 +0000 (14:31 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 30 Jan 2019 16:26:23 +0000 (17:26 +0100)
commit99aa1b4f26402921cae9b5c22436975be3854a98
tree7b3620299724133c5627bd9fb4121a3e6bb62755tree | snapshot
parent7fe0455177c5a1524b9c519c767f5f39a0c1db12commit | diff
Cleanup AbstractValueNodeModificationStrategy

The implementation did not log offending values and reused
verifyStructure() to perform validation, making analysis difficult.

Refactor the value check so that it is in a private method invoked
from the three codepaths which need to perform it, making code flow
clearer.

Change-Id: I315899982c0f304e0459d5ad33a19e56af0d2355
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 59227494c2ba2ff1543874b32e69cd5f44d9d51c)
yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/schema/tree/AbstractValueNodeModificationStrategy.java diff | blob | history
OpenDaylight YANG Tools