Code Review / controller.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 01578da) | patch
BUG-650: clarify TransactionFactory close state handling 24/11124/5
authorRobert Varga <rovarga@cisco.com>
Fri, 12 Sep 2014 21:45:02 +0000 (23:45 +0200)
committerRobert Varga <rovarga@cisco.com>
Sun, 14 Sep 2014 12:53:47 +0000 (14:53 +0200)
commit05910db0c56c6469e956400ceef8658b7d21ecf6
treeb906cc065b197988ccf8426abc8bafc070f41d6ftree | snapshot
parent01578dad303b97c2b3ac3fb5bb39d2be7367d21bcommit | diff
BUG-650: clarify TransactionFactory close state handling

getAndSet() translates to a get() + compareAndSet(). We already know the
expected value, so we can skip the first bit. The nice thing is that CAS
will readily give us a boolean, so the code becomes a bit more clear,
too.

Change-Id: Id77dcefb7ba85512fcebbcae7f43f08cd8094fbc
Signed-off-by: Robert Varga <rovarga@cisco.com>
opendaylight/md-sal/sal-dom-broker/src/main/java/org/opendaylight/controller/md/sal/dom/broker/impl/AbstractDOMForwardedTransactionFactory.java diff | blob | history
OpenDaylight controller