Code Review / yangtools.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: f75b0d3) | patch
Move checkNotSealed() to addBuilder()
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 28 Mar 2019 10:22:55 +0000 (11:22 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Thu, 28 Mar 2019 11:02:32 +0000 (12:02 +0100)
commitba8945f37f70d44315dcf95bdda19e852e3ab74f
treeb87fa6da93f80379e14b9779ec004f17d2c8a1b7tree | snapshot
parentf75b0d3b21487bf410ac9683ed160d080a6f776ecommit | diff
Move checkNotSealed() to addBuilder()

addBuilder() is called after previous check for seal from all callers,
who do not have any side-effects aside allocating arguments to
addBuilder().

Move the call to checkNotSealed() to addBuilder(), making the code
more straightforward and possibly preventing bugs from creeping in.

JIRA: CONTROLLER-1887
Change-Id: I127385378957deb7f38f1203870a0e52cd630644
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
java/org/opendaylight/controller/cluster/datastore/node/utils/transformer/NormalizedNodePruner.java diff | blob | history
OpenDaylight YANG Tools