Remove unused import statements in binding-generator-impl

Change-Id: I094a012432ddb68e75bd6a64dc55c48caa0dcb2b
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Remove unused import statements in yang-parser-impl

Change-Id: I23ac4c917da16dcd4190bd9f7c707e962d001eff
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Remove unused imports in yang-data-impl

Change-Id: I353e127afd929f230da9bd70cbe2d599e8e24ea6
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Remove unnecessary import statements in yang-data-api

java.lang are always implicitly imported.

Change-Id: I926516c3a731efe9d4770e379adb9e02b1d81ac5
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Bug 2532: Fix potential NPE in Utils.listFiles"

Introducing binding-parent

Change-Id: Ib10d4505ea3fe3238d029da3aebdabd9d3241397
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Bug 2532: Fix potential NPE in Utils.listFiles

Change-Id: I09fbcb8eaa98ea52e1c0ad09adc083c7a471c9a1
Signed-off-by: Ed Warnicke <eaw@cisco.com>

BUG 2468 - leafrefs via import statement

Adds method which search for base type of leafref in module specified via
QName value.

Change-Id: Ib1979b56604429fccac2d9955387085aa59ed01d
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Merge "Cleanup binding proxy instantiation"

Cleanup binding proxy instantiation

Profiling of BGP topology provider has revealed a performance bottleneck
when instantiating proxies for list children. As it turns out around 68%
of CPU cycles is being burned in looking up the proxy class and its
contructor. This patch performs the obvious tradeoff by caching it and
exposing a utility function to subclasses.

Change-Id: Icb76f232df8223925fa802859313bbf63ee58eeb
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Clarify NotificationListener purpose"

Merge "Clarify code flow in SchemaAwareApplyOperation"

Prune empty nodes from transaction when it is sealed

When we seal() a ModifiedNode tree, check also whether we have unmodified
children and prune them. Also turn a SUBTREE_MODIFIED with empty nodes
into an UNMODIFIED node.

Change-Id: I077d23023fdbfb4ab232ed826cc32b4541779e87
Signed-off-by: Robert Varga <rovarga@cisco.com>

Move InMemoryDataTreeModification.mergeImpl()

The merge implementation should really live on
OperationWithModification, so this operation routing is consistent. This
has the side effect of speeding up verification on merges.

Change-Id: I2c07fa65b5846f618a476cc6f7f9d0c862b85811
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Cleanup delete/merge/write cohesion"

Merge "Mark ModifiedNode as NotThreadSafe"

Merge "Fix checkstyle if-statements must use braces in yang-model-util"

Merge "Fix checkstyle if-statements must use braces in yang-parser-impl"

Merge "Fix checkstyle if-statements must use braces in yang-data-operations"

Merge "Fix checkstyle if-statements must use braces in binding-generator"

Merge "Do not access closed via updater"

Cleanup delete/merge/write cohesion

The three methods are not public, and are not used in fluent way. Group
them together and make sure they look the same.

Change-Id: Ib0acb09020b381e8fc259e862ed6b340bd7260e2
Signed-off-by: Robert Varga <rovarga@cisco.com>

Mark ModifiedNode as NotThreadSafe

The class is not intended to be threadsafe, so mmark it as such. Also
remove an invalid @GuardedBy annotation.

Change-Id: Ib2a5ee9710ff41447ff999d5056ae71d06b328a3
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Match InMemoryDataTreeModification.write() argument name"

Do not access closed via updater

This is a volatile field and an Updater.get() is equivalent to a read.
Use a direct read, thus making the field visibly used. This removes the
need for a warning suppression.

Change-Id: I0799ae80c2431b9d48bb5f62f681ed4fe5289f41
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix checkstyle if-statements must use braces in yang-model-util

Change-Id: I5535870da8d317999b10b4e07e79d849ccc531a4
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix checkstyle if-statements must use braces in yang-parser-impl

Change-Id: If6e41ded71028c7f506e87b310f1ea5b8cdc22af
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix checkstyle if-statements must use braces in yang-data-operations

Change-Id: Ia51055f22e2e3bc7693f35337c6d86e3d6663531
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix checkstyle if-statements must use braces in binding-generator

Change-Id: I929ee367d7263ba22eecd788d39688fd719bfd6b
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Match InMemoryDataTreeModification.write() argument name

This is an implementation of an interface method, so make sure the
argument name matches.

Change-Id: Ia2ee1d5e614aafac9d232abcefec65253c016bc9
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2470: fix ADD/DELETE state compression

If we are deleting a previously-modified node, we need check whether the
node existed before the transaction. It was there the modification needs
to be turned into a DELETE. But it this transaction introduced it, we
need to turn into a no-op -- otherwise we will trigger a wrong path.

If we don't the user will experience the effects of attempting to delete
a non-existing node, which is clearly not what the user intended.

Change-Id: Iceb926d69e1e9df359b3299e2e38a90ba3565fd1
Signed-off-by: Robert Varga <rovarga@cisco.com>

Improve readability of ordered checking

Rather than overwriting the default, move it to an explicit else path.
This highlights the fact the value is initialized precisely once.

Change-Id: Ic8340705d3c68d6b40b04467af69e4d6a717d9de
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "BUG-2470: implement InMemoryDataTree.toString()"

Merge "Bypass data tree generation on empty merge"

Merge "BUG-1914: bump API requirement"

Merge "BUG-2498: optimize enum's forValue() method"

BUG-2498: optimize enum's forValue() method

Use an ImmutableMap, as it provides optimized lookups based on
cardinality. Also hide the map from the package and turn it into a
constant.

Change-Id: I01591e56646705891fcc3fb8b871071169d35093
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1914: bump API requirement

This bumps the required maven API version 3.1.1 and removes the manual
override needed with previous versions.

Change-Id: If3b85169dccc9583ee93114c2b0c4f3fc9e17eb2
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bypass data tree generation on empty merge

A merge operation may end up not introducing any child nodes. Current
code forces a mutable/immutable cycle. This may end up copying maps or
creating a TrieMap snapshot -- which is not then modified and directly
reused.

This introduces a simple check and a shortcut, which just creates a new
metadata node.

Change-Id: I3fe4d07a139986d473a6bf55a2b40ff4350bd699
Signed-off-by: Robert Varga <rovarga@cisco.com>

Clarify code flow in SchemaAwareApplyOperation

When applying merge, we have a fall-through situation on the code.
Eliminate it by introducing a temporary variable. This makes the code a
bit more readable.

Change-Id: I3f468eb1235866ed627acdf5b60e6f890e44188a
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2470: implement InMemoryDataTree.toString()

For debugging purposes, we need a way to dump the current state of the
data tree. Implement proper toString() method.

Change-Id: I808b83b1e89eaa79d2a8d19b2e551884e7a3dc50
Signed-off-by: Robert Varga <rovarga@cisco.com>

Clarify NotificationListener purpose

This adds a comment about what the intended use of NotificationListener.

Change-Id: I84116fda4f7f68d63e8ad0e97365087df01b564a
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1914: force maven 3.1.1

Activates maven version enforcement.

Change-Id: I19bbd53e93de3ccbf029a2e029e479217e2c0c09
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix feature-test failing with surefire 2.18

feature-test fails maven-surefire-plugin 2.18 (but works with 2.17) due
to it assumming the constructor is called only once and modifying global
state. That global state is guarded from multiple modifications with an
Error. Move the offending initialization into static initializer, thus
ensuring we invoke the function only once.

Change-Id: I771ce384927769436ff0c8133471cefe27954a3d
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Remove xtend plugin binding-data-codec"

Do not use deprecated ClassLoaderUtils

Migrates to the class provided from the util package instead.

Change-Id: I988c4cc0d04062780ebd842ab596604b0c7cf287
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove PropertyPair

This holder is unused, so remove it, reducing clutter.

Change-Id: Ib808582faf11cfa9937e76a7d025ed2a66a5becf
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove xtend plugin binding-data-codec

The code does not use it, so save some time by not invoking it at all.

Change-Id: I633516da5cb152ea6c253a276d369bef002744ec
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Allow {Class,Method}Generator.process() to throw CannotCompileException"

Merge "Remove unused declarations"

Merge "Bug 2271 - NPE during generation of java binding for case list-grouping-choice"

Merge "Fix logging checkstyle"

Remove unused declarations

These properties are defined in odlparent, no need to repeat them here.

Change-Id: I34e05fa5bdf4d0937c967405c23d40c41c1b4187
Signed-off-by: Robert Varga <rovarga@cisco.com>

Allow {Class,Method}Generator.process() to throw CannotCompileException

This exception is naturally thrown by Javassist, so allowing our users
to not encapsulate it is a Good Thing.

Change-Id: Ibc558501e799a2c4c6599d6d7c5da668335c7270
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Add model parent"

Bug 2271 - NPE during generation of java binding for case list-grouping-choice

The SchemaContextUtil class does not find some types of schema elements nested
in other elements (e.g. groupings in groupings - Bug 2117, groupings in lists,
groupings in Rpcs, groupings in rpc outputs and inputs, groupings in
notifications etc...). Therefore complete rework of findNodeInModule and
related methods has been performed. This version of patch set (i.e. 5) works
also with yang modules containing siblings with the same name.

Change-Id: Ia5f9d437c05ace6d94999eac17308a756c439f82
Signed-off-by: pkajsa <pkajsa@cisco.com>

Merge "Resolved some sonar issues: If Stmts Must Use Braces"

Resolved some sonar issues: Hide Utility Class Constructor

Change-Id: I369df97a4de6c32dcb30a87516b0ec174368d38a
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Fix logging checkstyle

-allow literal string concatenation in log message

Change-Id: Iab33788f5db8e0e640c1f5d3a34494d84c2254d4
Signed-off-by: Milos Fabian <milfabia@cisco.com>

Merge "Introduce QNameModule.toString()"

Resolved some sonar issues: If Stmts Must Use Braces

Change-Id: I904bf50a98ba72f6887c540970a10b951c4f8beb
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Introduce QNameModule.toString()

Rather than using the default, let's add a ToStringHelper based
toString().

Change-Id: Ib9240564db8f710d36f9281054b5e2825d9790a0
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1953: perform proper string validation

Activates the use ot Patterns to enforce the string value has the format
specified by the yang file. Also fixes mutability of the exposed list,
which may have been attacked, injecting wrong strings.

Change-Id: I32d0ceb836df1f5ed04122a10115f66f75bfa1ec
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 106188585a33e10cd910acb16e1d18eba77c8268)

Merge "Bug 1677 - Parser: ListSchemaNodeBuilder keys needs to be a LinkedHashSet"

Do not use YangInstanceIdentifier#getPath()

Moves TreeNodeUtils away from using deprecated getPath().

Change-Id: Ida7edbd89b08d7365d30d20a1e9657a106886c10
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Bug 2157 - Race condition when adding a RPC implementation with an output"

Merge "Improve InstanceIdentifier.keyOf()"

Merge "Fix indentation"

Merge "BUG-2402: add FIXMEs and javadoc"

Merge "Reduce SchemaContext cache to weak values"

Improve InstanceIdentifier.keyOf()

Rather than throwing a ClassCastException, this documents that the
keyOf() method can throw an IllegalArgumentException.

Change-Id: I5d5b9e10c7329c6bc909b28b07c7bc6500cfcd5a
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "BUG-2304 Fix SchemaContextUtil for augmented nodes."

Merge "Bug 1781: Fix serialization of leafrefs"

BUG-2304 Fix SchemaContextUtil for augmented nodes.

Also keep backwards compatibility for invalid leafrefs
+ Fix the serialization in XMLStreamNormalizedNodeStreamWriter (btw. interesting name for a class)

Change-Id: I46cfcef46c97267c7e9b5bfebe03e42e9071e1e7
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Bug 1781: Fix serialization of leafrefs

- fixed search leafref node specified with relative xpath

Change-Id: I4b6eb6ab94b954ab5b28129dc332590d32b8a7c4
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

BUG-2402: add FIXMEs and javadoc

This adds an explanation about the interfaces being used and makes an
explicit warning for users to not use the exposed method.

Change-Id: I07753bea7380f7d92165b8ecb1840ee07d36dc06
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2401: remove LOG.warn() from entrySet()

As it turns out, entrySet() is called from
AbstractImmutableDataContainerNodeBuilder when it modifies children (via
HashMap(Map) contructor). Remove the warning and add explanation about
what is going on.

Change-Id: Ifbd99458c83d8afa25c3be7778fb28b579d4b2d8
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix indentation

Fixes wrong indentation and import ordering.

Change-Id: Ibb5593118a32b79d74edc97bed0ac2dceaa5b881
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Bug 2354: Fixed parsing of block comments between statement and argument"

Merge "Remove unused imports"

Merge changes Ic8e48ed1,Ie6bc9bbf,I3e94c637,I84c8ac42

* changes:
  BUG-2390: Use child size hints
  BUG-2390: Lower HashMap allocation defaults
  BUG-2384: improve ImmutableUnkeyedListNode efficiency
  BUG-2377: introduce internal UnmodifiableChildrenMap

BUG-2390: Use child size hints

This patch makes sure we pass down the sizing hints into the
implementation. This will allow us to make better decisions when
resizing maps.

Change-Id: Ic8e48ed108e20381aa0c91a5d8f7e3bb57acf180
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2390: Lower HashMap allocation defaults

This lowers the default hashmap capacity to 4, hopefully lowering our
overall overhead. It also introduces sizeHint arguments, to the maps can
be better-sized in the future.

Change-Id: Ie6bc9bbf79e7c5c63c3dd0000585660cbc09d368
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2384: improve ImmutableUnkeyedListNode efficiency

With moderately-loaded BGP instance we are using quite a few on empty
unkeyed list nodes. The fact they are empty means we waste about 8 bytes
per instance just tracking empty list. Introduce a dedicated class to
take care of this common special case.

Change-Id: I3e94c637f5d7d47d39d274bae96850307f5c439d
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2377: introduce internal UnmodifiableChildrenMap

This new class acts a replacement of java.util.Collection's version of
the same. Unlike the standard class, this one does not retain cached
versions of keySet and entrySet, since those are unused anyway. We can
also perform an instanceof check, which should speed up operations.
Finally, we check for map emptiness and use Collections.emptyMap()
when possible.

Change-Id: I84c8ac428d6b207fafccf8cc898f0555b104a4ee
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove unused imports

These imports are unused, creating eclipse warnings.

Change-Id: Id4e9a51131489ec426537a6af32ed77cd0ff161f
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG 2353 : LeafSetEntryBuilder does not compare byte array values correctly

Use Object.deepEquals to compare the value of the NodeIdentifier and the actual
value to be set on the LeafSetEntry

Change-Id: I4482b19d029058a0f78d3b19508d6f7797448710
Signed-off-by: Moiz Raja <moraja@cisco.com>

Reduce SchemaContext cache to weak values

While constructing a SchemaContext is potentially costly, we do not need
to retain them if they are not used for long periods of time. Soft
references attempt to retain the context until the memory pressure is
pretty high, while weak references are cleared as soon as the object is
proven to be weakly reachable.

Change-Id: I2df2a78218d3b439edadda2192ff541e4c1c0fee
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 2354: Fixed parsing of block comments between statement and argument

Lexer didn't contain BLOCK_COMMENT_MODE in VALUE_MODE,
therefore parser tried to parse comments instead of skipping them

Change-Id: I6d131a45df121570537686006aa31397124ea6ca
Signed-off-by: Martin Ciglan <mciglan@cisco.com>

Merge "Bug 2326 - NormalizeNode equals fails when NormalizeNodes being compared contain binary data which is identical."

Bug 2157 - Race condition when adding a RPC implementation with an
output

* added wait for schema in 'getRpcQnamesFor()' method

Change-Id: I2baa71f4237a1125db9a6e5ec8077237dc8ef544
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>

Bug 2326 - NormalizeNode equals fails when NormalizeNodes being
compared contain binary data which is identical.

Change-Id: If049e215156f5338eb3ff0ab95d19eecd622defd
Signed-off-by: pkajsa <pkajsa@cisco.com>

Merge "ISIS Yang model compilation issue"

Bug 1677 - Parser: ListSchemaNodeBuilder keys needs to be a LinkedHashSet

- changed type of kyes in ListSchemaNodeBuilder

Change-Id: I838d51b76f70f6446169f72407db8da43fb5ee03
Signed-off-by: Ladislav Borak <lborak@cisco.com>

BUG-2329 Add handling of anyxml nodes in XmlDocumentUtils with schema

Change-Id: I32fe0658aa495f98b75e804cec58ad9dcea02cf3
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "Bug 1781: Fix serialization of leafrefs"

Bug 1781: Fix serialization of leafrefs

- added functionality to read schema node which is type of leaf ref and
  points to leaf ref, identity ref, instanceidentifier or typedef node.

Change-Id: I150847b859ac0616c2cf160192bb712aea8202bb
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Add model parent

This patch adds an explicit parent pom which can be used by downstream
projects to build API bundles containing a yang model.

Change-Id: I37b9ef0a457679d1909a790b8cbd98edad64da39
Signed-off-by: Robert Varga <rovarga@cisco.com>

ISIS Yang model compilation issue

Change-Id: Ib4bc98ad68237ebea4eccf5522dfd79db88a7906
Signed-off-by: Reinaldo Penno <rapenno@gmail.com>

BUG-1956: bump java-concurrent-hash-trie-map version 0.2.23

Bump the version again, this time accounting for the late API change.

Change-Id: I95e947adcef587069ea3579da98b3f5d217a6e58
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 7955cb613c1338ac46e122aac5d71a4f2dc45cf3)