Merge "Bug 2361: Added prototype implementation using new SPI."

Bug 2361: Added prototype implementation using new SPI.

This prototype level implementation of module, namespace,
import, uses and container statement shows usage
of meta-level SPI in order to introduce support for
concrete statements.

Note that this is same SPI, which is to be used
to introduce new statement or extension support
directly to the YANG parser.

Change-Id: I5124a06669d48d3a32d585c4c75de5b2228f2eac
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Martin Ciglan <mciglan@cisco.com>

Bug 2874: Fixed JSON parsing of choice inside augment

Code of JSON parser was aware of choice being part of
augmentation, but small typo resulted in not putting
choice inside this augmentation.

Change-Id: I2b90517571e28843e6cc824306be8a864a7c5387
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Bug 868: Removed Composite Nodes from Yangtools"

Merge "Bug 868: Removed Binding to Composite Node codecs."

Bug 868: Removed Composite Nodes from Yangtools

Change-Id: I69d274d08dbe73900377a6715ac6c6f784e831c1
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Bug 868: Removed Binding to Composite Node codecs.

Change-Id: Ia54519c88870fada74bd2d7ce579c0e77d687157
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-2383: deprecate ModificationType.MERGE

The entire ModificationType enum is an accidental leak from the
implementation. The most pressing deficiency is that we do not want to
leak the merge operation, but report it either as a subtree modification
or a write.

Introduce a new type, LogicalOperation, which contains all the current
and use it internally. The new name also implies slight changes in how
the states are called:

ModificationType.UNMODIFIED maps to LogicalOperation.NONE
ModificationType.SUBTREE_MODIFIED maps to LogicalOperation.TOUCH

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

Merge "Reduce InMemoryDataTree logging"

Bug 868: Made sure Restconf client POC does not use CompositeNodes.

Change-Id: I64a940fa86cb0a0f4adccfde401b7073f7bbf9b8
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Reduce InMemoryDataTree logging

In setSchemaContext it logs the "Attempting to install schema contexts"
message to info which causes significant clutter in the karaf log - as
many as thousands of messages on startup and shutdown. I don't think
this message provides any useful information anyway so I removed it as
a similar message is logged to debug with context.

Change-Id: I0df9c3f49bd8173318b7b68584d066739ae7a562
Signed-off-by: tpantelis <tpanteli@brocade.com>

BUG 2868: Update iana-if-type with RFC 7224

Change-Id: Ide79db6c34fb23bd2c773a614c35d1e436b7d237
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Merge "Bug 2864: Fixed (de)serialization of leafrefs"

Bug 2864: Fixed (de)serialization of leafrefs

Leafrefs in JSON were not properly deserialized
for non-simple types as Instance Identifier
or Identityref, which requires schema-sensitive
parsing.

Reworked JSONCodecFactory to create and store codecs
by DataSchemaNode, since leafref resolution
requires location in data schema tree, which
was not required for other codecs.

Actual codec is selected by looking up schema of
type referenced by leafref and codec is constructed
from that type.

Change-Id: Ia650b99933b177d31bee3f4b38f42e22227120d6
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Bug 2857: Tied pre-existing implementation to BindingCodecTree."

Merge "Bug 2804: Fixed incorrect emiting of some arguments."

Bug 2857: Tied pre-existing implementation to BindingCodecTree.

BindingCodecTree provides public view into backing structure
of binding-data-codec, which was designed and already implemented
as data tree.

By exposing these power-users could build their own
Binding to DOM adapters, where they could customize some aspects
of serialization or precache over-used codecs in order
to speed up performance.

Change-Id: I827eb6014b28d0b6d4256b677ee56d3081ff80b0
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Bug 2804: Fixed incorrect emiting of some arguments.

For some YANG or extension statements, arguments
were not emmited as separate YIN element, but as
argument.

Added explicit check which extracts YIN element
information from RFC6020 mapping and ExtensionDefinition.

Change-Id: I8ebfdd21c2890e7eaeaf207ca43a7c4896369130
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Bug 2859: Key generation when anonymous Enum was used.

Binding list key was generated with incorrect type,
when anonymous enumeration was defined in leaf in grouping
and that leaf was used for list.

Change-Id: Idcb114be49280f77af8054d8d83c5d8e6f453665
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Bug 2846 - Ordered-by user is broken again - SFC is broken"

Bug 2846 - Ordered-by user is broken again - SFC is broken

* fixed gson codec to create ordered map node when "ordered-by user" is
defined in schema

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

Merge "Lower commons-lang3 dependency"

Merge "BUG-2512: get rid of commons-io"

Merge "BUG-2352: Remove RestMessageTools"

Merge "Migrate from BindingGeneratorUtil"

Merge "BUG-2829: make static QName field hold a cached reference"

Lower commons-lang3 dependency

This library is only used in tests, so lower the dependency.

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

BUG-2512: get rid of commons-io

Guava/JDK give us everything we need, no need to have yet another
library in the dependencies.

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

BUG-2352: Remove RestMessageTools

This is a prototype implementation, whose sole method never worked.

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

Add ietf-interfaces to model.

ietf-interfaces is a standard ietf yang model. This will be used by
vpnservice project.

Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: I787e945a39417cc85c79b11cb28264b82cd168e6
Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>

Migrate from BindingGeneratorUtil

BindingMapping is the new home for mapping module namespaces to java
package names.

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

Bug 2287 - TypeProviderImpl retains empty HashMaps

Allocating default HashMaps is wasteful, especially if we end up not
storing anything in them. This fixes that case by making sure we
allocate a map only after we are sure we are going to put something into
it. Also be conservative about the size, so we do not waste too much
space.

Change-Id: I85ad1506e732a5c3728ef3529e7e0b5b430d758a
Signed-off-by: Ladislav Borak <lborak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Bug 2818: Updated implementation of parser."

Bug 2818: Updated implementation of parser.

Moved special-case handling from nested classes up
to main parse cycle, which simplifies code. Also
added additional test, which tests behaviour
according the draft-lhotka.

Change-Id: I79bfa62221a6bbe914158b0fae3f51f4571ff9dd
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-2829: make static QName field hold a cached reference

A QName (and QNameModule) exposes an API to obtain a system-wide
shared reference to an object. Use this facility to share QNameModules
across all generated classes, as well as sharing potentially duplicate
QNames.

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

Merge "Eliminate all trailing whitespace in xtend"

Merge "Make sure we do not generate trailing whitespace"

Merge "Bug 2818: Introduced optional array for parsing root list items."

Eliminate all trailing whitespace in xtend

The API generator output should not contain trailing whitespace --
cleanup the templates to make sure we do not pollute output.

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

Make sure we do not generate trailing whitespace

If we get an empy line in description, it is possible for us to generate
trailing whitespace. Make sure we don't do that and also clean any
trailing whitespace from the source file itself.

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

Bug 2818: Introduced optional array for parsing root list items.

Specification draft-lhotka-netmod-yang-json-02 clearly
specifies that list must be serialized as array of objects,
but currently only user of yang-json is
draft-bierman-netconf-restconf-02, which specifies
and provides example where such list exists are also used
without arrays when parsing.

Introduced support for this case (and unit test),
in order to interoperate with clients, which follows
serialization as was provided in examples of restconf.

Change-Id: I4ebf7cfba17752de2184e9b7ac1d18c1ecac6f80
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Bug 2248 - mvn clean install is failing due to checkstyle"

Merge "Fixed generation of UnionBuilders for anonymous unions."

Introduction of more powerful binding codec APIs

Introduces new more expresive APIs, which allows
user to customize serialization / deserialization.

Introduced new concepts:

 - Subtree codecs
    - Tree and cursor based APIs for locating subtree
 - Caching Codec - codec which uses cache for serialization

Change-Id: If4868c9cd20c56d2ce0498bd2ded90b1265f7b67
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Fixed generation of UnionBuilders for anonymous unions.

Change-Id: I18e90e7d7227efe31767734df1ce525df60104d5
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Bug 2248 - mvn clean install is failing due to checkstyle

- added check in BaseTemplate.imports() if imported type has same package
  as generated type

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

BUG-1513: make sure client code remains compatible

This updates the single implementation of ChoiceNode so that it
implements the legacy interface, too -- thus instanceof checks will
continue to work.

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

BUG-1513: finish migration of ChoiceNodes

This migrates the leftover users to make them work correctly.

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

BUG-1513: introduce ChoiceSchemaNode

This introduces the alternative name, which does not clash with
yang-data-api. Also deprecated ChoiceNode.

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

Merge "Migrate to BasicGenerator"

Merge "BUG 2282 - JSON top level element without module name"

BUG 2282 - JSON top level element without module name

Json top level element now doesn't have to contain module name prefix.

If namespace of some child element was changed and only one child with
specified name exist in schema then its namespace is used by default.

If more then one potential child in various namespaces exists then
exception is raised. Exception is also raised if no such element as
specified in JSON input exists in YANG schema.

Change-Id: I8a38fde4f2e9b79562f6bf7c79acafd51c7bff32
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
(cherry picked from commit 6eee2a1ce9e69a7b653a7b4427573eff67588d1f)

Migrate to BasicGenerator

This is just a documentation reference, so update it to the new
interface.

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

Merge "Remove unneeded annontation"

Merge "Fix a generic warning"

Merge "Execute yang-maven-plugin"

Remove unneeded annontation

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

Fix a generic warning

Add generic arguments to fix a warning.

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

Add support for attributes int NN stream writer(list entry)

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

Execute yang-maven-plugin

This eases integration with eclipse, as we should be interacting with it
well now.

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

AnyXml NormalizedNode wraps a DOMSource value

Instead of wrapping a CompositeNode, a DOM source is used as inner value for
AnyXml. XML -> NormalizedNode parser was updated as well as NormalizedNode ->
XML stream writer.

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

Add support for attributes int NN stream writer

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

Merge "Bug 2766: Fixed parsing and serializing XPath Instance Identifiers"

Introduced MethodHandle based invokers for Notification and RPCs

This invokers are used to provide genereralized access to any
generated callback / rpc method using one approach for
any RPC.

Change-Id: I4d6ce7c71f658b5e2c6e805479aa47a8f09fb989
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Bug 2766: Fixed parsing and serializing XPath Instance Identifiers

Original implementation of Instance Identifier codec used
Splitters which did not have enough information to determine
that '/' is used in string literals.

Fortunatelly Instance Identifier is only very constrained
subset of XPath, so it was still possible to reimplement
correct token-based parser using Guava.

Other problem fixed with  parsing instance identifiers was
that YangInstanceIdentifier also contained components,
which are not present in XML / JSON serialization.

Change-Id: I4186f59f8e16c92eb2e0ca2bc0a6c12e7215a3be
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Revert "AnyXml NormalizedNode wraps a DOMResult value""

Revert "AnyXml NormalizedNode wraps a DOMResult value"

This reverts commit c5bde84eced27e2d073554420c3f47eca85af32c.

Commit was merged before clients were able to migrate.

Change-Id: I7ab48a36f5dbbd52e8b91be8ea049b72a776ea15
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Make sure maven artifacts are provided"

Merge "Remove unnecessary casts"

Merge "Remove unnecessary casts"

Merge "Remove unnecessary casts"

Merge "Do not rely on transitive dependencies"

AnyXml NormalizedNode wraps a DOMResult value

Instead of wrapping a CompositeNode, a DOM result is used as inner value for
AnyXml. XML -> NormalizedNode parser was updated as well as NormalizedNode ->
XML stream writer.

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

Preserve backwards compatibility in DomToNormalizedNodeParserFactory

Restore the former constructor with only CodecProvider but make it deprecated

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

Make sure maven artifacts are provided

We expect these to be provided, not brought in explicitly.

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

Remove unnecessary casts

xtend 2.7.x makes instanceof check an implicit cast for subsequent
variable usage, so these generate warnings. Also remove use of
deprecated types and speed up stringuilder usage by passing characters
instead of strings where possible.

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

Remove unnecessary casts

xtend 2.7.x makes instanceof check an implicit cast for subsequent
variable usage, so these generate warnings.

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

Remove unnecessary casts

xtend 2.7.x makes instanceof check an implicit cast for subsequent
variable usage, so these generate warnings.

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

Do not rely on transitive dependencies

Maven core and build API should have scope=provided.

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

XML -> NormalizedNodes parser identity-ref/instance-id/leafref support

The parsers now pars leaves of mentioned types properly.

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

Add AnyXml support to NormalizedNode parser

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

Merge "Remove unneeded build declarations in features"

Add rpc support to NormalizedNode writers

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

Remove unneeded build declarations in features

features-parent defines all the things we need, so the entire build
section can go.

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

Add more NormalizedNodes.findNode() alternatives

In some calling contexts we already have an optional and/or do not have
the path encapsulated in YangInstanceIdentifier.

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

Use Collection.size() instead of Iterables.size()

Now that NormalizedNodeContainer returns a Collection of children, we
can use its size() method directly, speeding things up just a little
bit.

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

Merge "Make sure NormalizedNodeContainer uses Collection"

Merge "DataContainerNodeBuilder should take a Collection"

Merge changes Ie600f25f,I99deeb57,I63bddcfe

* changes:
  Cap LazyCollections list
  Introduce UnmodifiableCollection
  Expand known immutables

Merge topic 'stable/helium'

* changes:
  Optimize getChild()
  Fix OrderedNodeContainer javadoc
  Fix ImmutableOrderedLeafSetNode.size()
  Fix EmptyImmutableUnkeyedListNode.getChild()

Make sure NormalizedNodeContainer uses Collection

Instead of an Iterable, specify a collection, as that makes the number
of elements known.

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

DataContainerNodeBuilder should take a Collection

List is too specific a collection to pass down into the builder. We care
only for it be Iterable, so a Collection is a reasonable middle ground.

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

Cap LazyCollections list

When we move from a singleton to an ArrayList, specify the size as 2, as
the arraylist can grow beyond that, but will not waste space if it
remains at two elements.

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

Introduce UnmodifiableCollection

This is a utility class which recognizes itself and Guava's
ImmutableCollection subclasses during contruction, preventing unneeded
encapsulation.

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

Expand known immutables

Void and Boolean are immutable types, too. Also use ImmutableSet instead
of a hown-grown utility.

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

Optimize getChild()

The classes are so they can access children directly. This saves
instantiation of unmodifiable view and checks related to that.

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

Fix OrderedNodeContainer javadoc

Documents the generic argument and fixes a typo.

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

Fix ImmutableOrderedLeafSetNode.size()

The method always returned 0, which seems to have been missed.

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

Fix EmptyImmutableUnkeyedListNode.getChild()

The interface contract states that we should throw
IndexOutOfBoundsException rather than returning null.

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

Remove unneeded dependencies

slf4j bridge should no longer be necessary, so remove it, along with
dependencyManagement override.

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