Code Review / yangtools.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 9c2cb3f) | patch
BUG-6497: Do not lose augmentation statement order 75/44875/3
authorRobert Varga <rovarga@cisco.com>
Tue, 30 Aug 2016 16:41:48 +0000 (18:41 +0200)
committerRobert Varga <rovarga@cisco.com>
Wed, 31 Aug 2016 09:39:58 +0000 (11:39 +0200)
commit3c4bdfac3a8b04ecd1d6eab2cdadbb365b54664e
treea8723ece21aed2624252f88a0dcf700ee78d315ftree | snapshot
parent9c2cb3f62c4edc8c0c3db7e74a7b46ac6e4b2b1dcommit | diff
BUG-6497: Do not lose augmentation statement order

Using an interim HashSet can cause unpredictable order
of augmentations, which means the binding spec will not
assign consistent mapping at runtime.

Fix this by using a LinkedHashSet, which retains insertion
order.

Change-Id: I503102d8b61453cc9c8d35b07158cd97138cda42
Signed-off-by: Robert Varga <rovarga@cisco.com>
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/AbstractEffectiveSimpleDataNodeContainer.java diff | blob | history
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/ChoiceEffectiveStatementImpl.java diff | blob | history
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/NotificationEffectiveStatementImpl.java diff | blob | history
OpenDaylight YANG Tools