Code Review / yangtools.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 37962d8) | patch
BUG-6497: Do not lose augmentation statement order 77/44977/1
authorRobert Varga <rovarga@cisco.com>
Tue, 30 Aug 2016 16:41:48 +0000 (18:41 +0200)
committerRobert Varga <rovarga@cisco.com>
Thu, 1 Sep 2016 08:18:52 +0000 (10:18 +0200)
commit6d051fb473374c133474d2e90aa68663ae948763
treea479f8b2fbacf1274915ee6f57e39c643dc5ef58tree | snapshot
parent37962d89fbd4032653b3918921181377c855b598commit | 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>
(cherry picked from commit 3c4bdfac3a8b04ecd1d6eab2cdadbb365b54664e)
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