Bump to odlparent 4.0.5

Change-Id: I4b122ff9c55c23df83440b3aa307fe276b56aade
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add karaf distribution

This adds a karaf distribution for the purposes of validating
upstream (odlparent) packaging.

Change-Id: I0c01ebad2cad2e34407eae139cb6b2a2dc119d51
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump odlparent to 4.0.5-SNAPSHOT

This bumps the odlparent version to latest developent,
fixing issues with distrubution (relative to 4.0.4).

Change-Id: I8d238e4bda2ddf2cfd241a7f5b65f5d08423a224
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump to odlparent 4.0.4

Change-Id: I0e9737eb596296ec70f754492911f51241927c9c
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Intern empty leaves

This patch teaches LeafInterner to intern leaves holding Empty,
as they have only one possible value.

Change-Id: I5afa161ec49b02c33a3e7be28bcda135b6d796cb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Expand leaf interner

This expands LeafInterner to also handle int8/uint8 types.

Change-Id: Ic9632c170252c88f76a7a7d557c783ee7d982701
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Mass migrate fields to use Immutable collections

We are using immutables in all but declaration. Help performance
a bit by using base classes.

Change-Id: I8408ec89667b747da264b13a3425bfc752cc4ef4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump versions to 2.1.6-SNAPSHOT

This is the start of next development cycle.

Change-Id: If0f7f75ac7560de6c976b797bd5e4b973ff1dfe4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Preserve attributes when parsing AnyXml from XML

So far, anyxml nodes have been parsed in following pipeline:
 String -> DOM -> Stax -> String -> DOM
Now the parsing is simplified to:
 String -> DOM -> Stax -> DOM

Remove the unnecessary "back to string" step. This step is not just
unnecessary but also stripes down attributes and namespaces from AnyXml
node value.

JIRA: YANGTOOLS-923
Change-Id: If13ce9c7397c37b96a532090e67683a635ae14ef
Signed-off-by: Jakub Morvay <jmorvay@frinx.io>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize SingletonSet.containsAll()

Use a single expression return to trim down some bytecode.

Change-Id: I49ca88033d36994de761abaa7eb28438e10a37e6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize ImmutableOffsetMap a bit

Use fluent StringBuilder, assign-and-compare and eliminate an
unneeded variable to trim down bytecode size.

Change-Id: Ib08a65e33759920e9a86751fe5b833052d8bce87
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Optimize ConstantArrayCollection.equals()

Use a single expression return, trimming the resulting method
size.

Change-Id: I5300ed93cc97de9b3b60ce885b6b8a13dd0c6927
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Allocate new keyes map lazily

Do not allocate the map for new mappings until it is actually needed.
This introduces some additional complexity via null checks, but that
is not something major.

Also refactor large equals() method into smaller chunks, so that it
can be inlined more easily.

JIRA: YANGTOOLS-919
Change-Id: Idb4b7a4cb7cde09af2eab0bcebfb71083e64ff2f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump version to 2.1.5-SNAPSHOT

This starts the next version development.

Change-Id: Ib2b1b5e8a8c33da1164ac7c193958650b47c0aa0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add predicate-free NodeIdentifierWithPredicates constructor

This saves some time and figuring out to users, as they can just
not specify any predicates and get a free list. This side-steps
some of the code done in the Map-taking constructor.

Change-Id: If06c50aad3c2918cc6ec150b4185837557ac3f40
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor ImmutableMapEntryNodeSchemaAwareBuilder

This builder's logic is not entirely efficient, as it can constructs
unnecessary NodeIdentifierWithPredicates purely for checking purposes.

Furthermore we can side-step child validation when we are switching
by available children, as we have the entryset available.

JIRA: YANGTOOLS-917
Change-Id: I57d0a156ced3abce7e61403c2b390c074850d612
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Factor out {SharedSingleton,ImmutableOffset}MapTemplate

Exposing the single-entry map case has the advantage of providing
instantiateWithValue(Object) method, side-stepping the need for
a temporary array. Split it out of ImmutableMapTemplate, so that
users can provide customized handling as needed.

The same holds true if the keyset is known to contain more than
one key, in which case using ImmutableOffsetMapTemplate can be
faster.

JIRA: YANGTOOLS-917
Change-Id: If08f492ff59bc89f3826a4122cfbcf34657c2210
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Improve ListEntryNodeDataWithSchema

ImmutableMapTemplate allows us to more efficiently instantiate
NodeIdentifierPredicates, as it takes care of ordering of keys
while maintaining an efficient internal representations.

It furthermore provides a keySet() method, which is useful for
quickly checking if a leaf is part of key definition -- unlike
ListSchemaNode.getKeyDefinition(), which is a List and therefore
must be linearly searched.

It also allows us to skip HashMap allocation for unkeyed lists,
clearly showing that the class should be split into two.

JIRA: YANGTOOLS-917
Change-Id: If2d7bbbc5c4fe1ca96097f374f4313aabb411908
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add ImmutableMapTemplate

We have couple some hot paths where we know the keySet of an immutable
map we want to instantiate. For these, using
ImmutableOffsetMap.*copyOf() is wasteful, as it performs unnecessary
cache looks and map lookups.

ImmutableMapTemplate can be used to amortize these preparatory steps,
so that an ImmutableOffsetMap (or SharedSingletonMap) can efficiently
be produced by either transforming values of some other map, or by
supplying values in the correct order.

Since we are in the area, improve documentation and @NonNull
annotations of both ImmutableOffsetMap and SharedSingletonMap.

JIRA: YANGTOOLS-917
Change-Id: I8e7808b9bfe751885b842715f7e0346c0e7485b2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Separate out ConstantArrayCollection iterator

This splits it out into its own static class.

Change-Id: I94a86c65eefdba79db2f5ec3a01b13dbad0b41b0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add EmptyDequeTest

This tests the class to 61%.

Change-Id: Iaf1f4dc14a31937a53a47766c7c42f05b423117d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Retrofit SchemaContextProvider

Since we already have a getSchemaContext() method, we can
get it documented and widely available for free.

Change-Id: Ib5aaa3d1d503a000a997beaee3d31002755e5998
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make InMemoryDataTreeSnapshot implement SchemaContextProvider

Note DataTreeSnapshot's duplication of getSchemaContext(), which
is indicative of a coupling to yang.model.api. This is fine for now,
as this coupling already exists through xpath interfaces, but it
needs to be revisited and cleaned by either removing the method,
or making it a non-default (and inherited from SchemaContextProvider).

Change-Id: If335f178e9e086b8b5185044a06d4296f81b2a74
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor XMLStreamWriterUtils

This refactors XMLStreamWriterUtils, so that it can be a proper
SchemaContextProvider. This allows us to propagate this property
through SchemaAwareXMLStreamNormalizedNodeStreamWriter the same
way.

Change-Id: Ibe0832e76d3e66ca27f8d345f7fd6f890b92f35e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Reuse writer field from parent

Writer is a duplicate field, which is already available as
getParent(). Remove the field and use the getter.

Change-Id: I1d5dd866512479268e32affb0ded301fadb5cb11
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Speed up UnorderedMapMixinContextNode

NodeIdentifierWithPredicates recognizes ImmutableMap, make sure
we remain on the faster path of it.

Change-Id: I95d745d6e7ba54dcdb422fac19256112a21f413f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup Immutable documentation

This is a simple formatting update, with small rewording to use
better language.

Change-Id: I51848c3a956c7251bad98418cac40ea0ffa943c2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Document Identifiable interface contract

Somehow this core interface went undocumented, fix that and also
clarify it's companion Identifier contract.

Change-Id: I009ed90e548770b46d636e5e41c07877642682d5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix HashCodeBuilder Eclipse error

With .build() required to return a @NonNull, make sure we match
the annotation.

Change-Id: I1baedf6af702ec77c053f209fe43ad75f357bccd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Properly propagate BuildContext

YangToSourcesProcessor was not properly propagating BuildContext,
fix that.

JIRA: YANGTOOLS-915
Change-Id: Id4f28fe640bbd4d4ed065b9aa62f0becb2db43ec
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add OptionalBoolean.ofOptional()

This adds a simple utility method to create symmetry with
fromOptional(). Also cache TRUE/FALSE optionals, so accesses
get a bit faster.

Change-Id: Ife95d26f9b58e6859991d544eda52d0094302c30
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump versions to 2.1.4-SNAPSHOT

This starts the next release development.

Change-Id: I4035fb4a68b6778151fbb5aac020a43af4505ca2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fixup xpath versions (again)

This is the second time I missed the version bump, let's fix it
up.

Change-Id: I834b2ab339c2b1ebc974d8af0ac7e0a9a50a721e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Update RpcResultBuilder.buildFuture()

Returning a FluentFuture here is extremely useful, retrofit
the builder by introducing a hidden superclass which returns
ListenableFuture, which allows us to seamlessly return
a FluentFuture instead by forcing the compiler to emit a synthetic
bridge method.

Change-Id: I3d9725c666ff5d78f795eb0eada6e7acf81e8e70
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Shorten Prerequisite reference

We have the interface imported, no need to refer to it via
ModelActionBuilder.

Change-Id: Ife9f840bffe430264cd7b9ba793f604a0dbc4e5c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Tolerate empty instance identifier in ImmutableNodes.fromInstanceId()

If the user specifies an empty InstanceIdentifier, we fail with an
undocumented NoSuchElementException. Make sure we properly handle
this case by returning a container which matches SchemaContext.NAME.

JIRA: YANGTOOLS-914
Change-Id: I0b503d3bed99d73d88e037d35979c503215f3b52
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix StmtContext nullness problems

Introduce coerceParentContext() and coerce(Raw)StatementArgument(),
which are guaranteed to return a non-null reference or throw
a VerifyException. Convert users pointed out by Eclipse nullness
analysis to use these instead their @Nullable counterparts.

JIRA: YANGTOOLS-907
Change-Id: Ib6d80a9597ec8437194dc553ea71445599ee381c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate SourceException and its subclasses to JDT annotations

This is straightforward change, just switch over annotations and
fixup some of the warnings in the corresponding test.

JIRA: YANGTOOLS-907
Change-Id: I9d757381d71af205f3ed4a41778a2027a66d3fdb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Limit the TCCL when instantiating UntrustedXML

JAXP newInstance() methods can end up going through ServiceLoader
with TCCL as its ClassLoader, which means our constants may end up
pointing to a ClassLoader beyond our normal visibility.

This means we can end up picking up a different factory depending
on where we are called from and also can mean the TCCL cannot be
garbage-collected.

Force the TCCL to be the ClassLoader holding UntrustedXML, thus
establishing tight coupling between JAXP implementation discovery
mechanism and UntrustedXML.

JIRA: CONTROLLER-1867
Change-Id: Icacb0e193c7360c73bd07bc5c815bf803ca9ac44
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 6fd73a016d19cfb336ab12f6eddfc2d5934fb3fc)

Fixup errors reported by null analysis

This fixes up nulls being used in illegal contexts.

Change-Id: Icba09ff95e0c8a41615b0925de8f12006ea92059
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert parser-{reactor,spi} classes to JDT annotations

Migrate away from JSR-305 annotations to JDT annotations.

Change-Id: I268808a6da501eedce9bc34df0d9c5f70b145230
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Undo yang-common JDT annotation in parameters

Using @NonNull for arguments to construction methods causes massive
amounts of warnings downstreams, disable them for now.

Change-Id: I940793d9ec0a899c04027d947f129130c88634d6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make Delegator.getDelegate() non-null

The implied intent is that the delegate is always non-null. Fix this
up.

Change-Id: I29f21eeafd724ee3799db736c9a4bbb6bc4f1e3d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate yang-model-util annotations

This moves over to use JDT annotations instead of JSR305.

Change-Id: I98d70e53a951d41c3533272da016fb8afa223a49
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate yang-parser-api annotations

This patch migrates nullness annotations from JSR-305 to JDT.

Change-Id: I23fe36ffbf3a5929c48239bbdcfd9ef1e62ae0b6
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate XPath-related interfaces to JDT

Rather than using JSR305 annotations, use JDT equivalents.

Change-Id: I76025f02f74a326fef1e72b6fb768897b68b33f7
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate model.repo.api to JDT annotations

This mass-migrates all of model.repo.api interfaces and classes
to use JDT nullness annotations.

Change-Id: I740d22b348d9f12cc30389e8f569b0dc912e7a57
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate SchemaSourceRepresentation annotations

Migrate from JSR305 to JDT annotations, along with implentations.

Change-Id: I38c8c567a91c42e0c4387cfc3d7baf06cf495e2b
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate StatementDefinition @Nullable

This is of @Nullable is rather trivial, update it to JDT annotation,
and update implementations to eliminate warnings comimng from Eclipse.

Change-Id: Ifd3fea2f3e9fe68a51c0599b11889ee86b3ef7c6
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup IdentifierNamespace annotations

Migrate from javax.annotation to JDT annotations, cleaning up
implementations while we are at it.

Change-Id: I0147422c3ac2353c08f8d810906974922b8a6710
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate EffectiveStatement to @NonNull

This migrates @Nonnull to @NonNull, eliminating the need to
annotate implementations.

Change-Id: Ia7527df07e5ea4df839f8a6051dbc5bd2280aa66
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate TypeDefinitionAware to JDT types

This migrates all implementations of TypeDefinitionAware to
JDT @NonNull annotation.

Change-Id: If3a083835f1be90063a62c3241df4f6970c97442
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate statements to JDT annotations

These are trivial conversions from javax.annotation to JDT's
NonNull.

Change-Id: If3f6fed71917101febb2bce0d1b2a853668f205a
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add CheckedBuilder

API design can call for a specific exception to be thrown when
a builder is not properly initialized. Add CheckedBuilder which
can throw a specific exception.

Change-Id: I03fcc7a96363d40217fc7e363535fffe89699da0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate ActionStatementContainer to JDT annotations

This is a trivial use of @Nonnull, migrate it over to @NonNull.

Change-Id: Ie2167fa3574441fbb859b4ea5c56fdefe3b1b264
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate RevisionStatement to JDT annotations

This is a trivial use of @Nonnull, migrate it and eliminate useless
override in RevisionStatementImpl.

Change-Id: I096627189a3d37a49b612ee8ea7cc544570a8eae
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate AbstractDeclaredStatement to JDT annotations

Baseline yang-model-apis have been migrated, move over to JDT
annotations here, too.

Change-Id: I8de262e3730ec616b2158dc25f4bbe4171a37e70
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate SchemaNode to JDT annotations

This is trivial use of @Nonnull, migrate it over along
with implementations, also cascading annotations to SchemaContext
and SchemaPath.

Change-Id: I7359582d6696f60a262b779d9aed728e4f828637
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate SchemaContext to JDT annotations

This is a trivial use @Nullable, migrate it and strengthen
annotations around default methods.

JIRA: YANGTOOLS-907
Change-Id: I8bbbe0e80bfe5037564913099f14ffd10d5df583
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate {Action,Notification}NodeContainer to JDT annotations

This is trivial use of @Nonnull, just migrate over to JDT.

Change-Id: I352516b8bfcf6443118ed0a4eb1280793972059d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate UsesNode to JDT annotations

This is trivial use of @NonNull, migrate it over, along with its
only implementation.

Change-Id: Iab8c40fe5454718db0ac440f03c07b45dcee1806
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate yang.model.api.type to JDT annotations

This is trivial use of arguments, migrate all interfaces and
internal utility methods to JDT annotations.

Change-Id: Ic95d7051aa19b4f88d994da5c9586b7fa3a75ef7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate ModifierKind to JDT annotations

This enumeration is using @Nonnull in trivial ways, migrate it.

Change-Id: I8eab974c4d8dbda5c168bd0477c501426f906473
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate UniqueConstraint to JDT annotations

This migrates the simple use of @Nonnull to JDT use, guarding
the implementation with verifyNotNull().

Change-Id: I4310fffd0c1934668ad9dc14b1675694a51bb722
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate EnumTypeDefinition to JDT annotations

This is a simple use of @Nonnull, migrate it over along with
all implementations.

Change-Id: Ide74c244134bbe4be62d74945059de486c57bed5
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate ListSchemaNode to JDT annotations

This migrates the trivial use of @Nonnull to JDT's @NonNull.

Change-Id: I08c54870093c9e41ea1b5da2a16b6be90921a4d0
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate LeafListSchemaNode to JDT annotations

This is a simple use of @Nonnull, migrate it to JDT.

Change-Id: Iad55cc0536a43a407c336cd68886e8d24927544a
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate ElementCountConstraint to JDT annotations

This is confined use of @Nullable, migrate it.

JIRA: YANGTOOLS-907
Change-Id: I948d3523aee4088109b1809c2309e13af7efc2c3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate IdentitySchemaNode to JDT annotations

Rather than using JSR-305, migrate IdentitySchemaNode and its
implementation to JDT annotations.

Change-Id: I7e1ca793c86ca6a2576292cf0791dcaa17a5d346
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate BitsTypeDefinition to JDT annotations

This migrates @Nonnull to @NonNull in the interface and all its
implementations.

JIRA: YANGTOOLS-907
Change-Id: Ic4a43b8e1e6a85894995c1c73032490111079874
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate DocumentedNode/Status to JDT annotations

Switch DocumentedNode and its implementation to JDT nullable
annotations.

JIRA: YANGTOOLS-907
Change-Id: I7eea3429d1ee7617f0731c7416456a44e2caffbe
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch DataNodeContainer to JDT annotations

We are using a single @Nullable, switch that over.

JIRA: YANGTOOLS-907
Change-Id: I154822f057850ab1c5a65c62be01a6ab04c35c72
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove javax.annotation nullness from yang-model-export

Switch to using JDT annotations instead and remove bundle plugin
exclusion.

JIRA: YANGTOOLS-907
Change-Id: I708076719fe4e91049946c49640459fc9baa215c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove javax.annotation use from yang-data-transform

Switch to JDT annotations and remove the bundle exclusion
instruction.

Change-Id: I487d384cba31b9337c5291373df5a6410278070b
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup StmtContext nullness annotations

Switch @Nonnull to JDT's @NonNull, propagating results to
implentations.

JIRA: YANGTOOLS-907
Change-Id: I44bd6791f23a7e3dfb6864cba15763713022a327
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Update (Abstract)StatementSupport nullness annotations

Use JDT annotations instead of JSR-305.

Change-Id: Ica49e1c54aba95677fb883c01d4040e4bee44c8b
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup nullness in StatementSourceReference

getStatementSource() must return non-null, make sure we express
it in the contract.

Change-Id: Ifa098ce7bda78878ad5a333849c1c254f111dfbd
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup StatementFactory Nonnull annotations

Switch to using JDT, all implementations are ready for this.

JIRA: YANGTOOLS-907
Change-Id: I26870cea034d50fff44c3af1091d87906eb5b6a5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Cleanup ModelStatement/StatementDefinition non-nullness

This patch removes @Nonnull in favor of @NonNull, adjusting
implementations to remove Eclipse warnings.

JIRA: YANGTOOLS-907
Change-Id: I6d04dd3d807316f8cc39fe3481e0252c20673460
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add CopyHistory.toString()

This method makes for easier debugging, add it.

Change-Id: Ib25dc0414dce2ddacecaedafb14b2e0d42e3d50d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add a unit test showing augmentation handling

There is a world of difference between augmentations added to
a node vs. instantiated nodes. This patch adds a test which shows
it.

JIRA: YANGTOOLS-911
Change-Id: Ida3ce1fab8db09470e09d101de740b9b7d27a687
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add yang.common.Revision.MAX_VALUE

Since Revision is Comparable, it is useful to capture its maximum
value, so it can be used within Ranges composed of Revision and
some other Comparable.

Change-Id: Ib2e0dba348dc5cbc53b19bc4ae6d7062c6feabb6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Attach error listeners to lexers

We currently disconnect parser from standard output, but lexer
may incur errors as well, hence we should do the same dance to
lexers, too.

Change-Id: Ib901cb1fbdba05c721aeba5d60a5e8bda5f48087
JIRA: YANGTOOLS-905
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Exclude javax.xml.stream dependency

This dependency is satisfied by the JDK, hence we do not need to
pull it in.

Change-Id: I440114204f064239e3e25215e53fe67981ddf6d7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump versions to 2.1.3-SNAPSHOT

This starts the development of 2.1.3.

Change-Id: I710c410311cff014ed4249a158242400be3c3a62
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fixup MinMaxElementsValidation with disappearances

When a list disappears, even when it has a certain number of minimum
elements, MinMaxElementsValidation should not take any action.
The resposibility for enforcing the list being present when it has
min-elements > 0 falls under perview of MandatoryNodeValidator, which
covers the non-existence of the list at the proper enforcement point
-- something MinMaxElementsValidation cannot do.

JIRA: YANGTOOLS-909
Change-Id: I3612a1c1461456b2bd41651decab1032581f85d1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

MinMaxElementsValidation needs to deal with lists disappearing

Constrained lists may become empty, in which case applying
the modification will result in a non-present TreeNode. In that
case the list needs to count as zero elements rather than tripping
up VerifyException.

MinMaxElementsValidation really needs to become a full-blown proxy
ModificationApplyOperation instead of being piggy-backed on
SchemaAwareApplyOperation, as otherwise can end up tripping up
preconditions which ware dealt by the proxy.

JIRA: YANGTOOLS-909
Change-Id: Ia410231f701223792a821cd69af84b7742fe5156
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Drop the use of project.groupId

Spelling out org.opendaylight.yangtools makes things more
clear without being too verbose.

Change-Id: I10aabbd4a07eb6d6c25aa8f68157a889ee7ab43e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Add bundle-parent and use it

This concentrates common version definitions into a yangtools
bundle-parent and reuses it them where possible -- shaving almost
500 lines of XML.

Change-Id: Ifc79fee557dc0a49fc0a277c2fafa34199171151
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

jaxb-api should be scope=compile

Keeping scope=provided breaks downstream tests with JDK9+,
hence we need to rework the dependencies to make sure we always
pull jaxb-api, but we also need to exclude it from odl-yangtools-data
feature, because it is provided by Karaf.

Change-Id: Id09281c89a74c3d6c780179dfa21ced545318ba1
JIRA: YANGTOOLS-906
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate parser extensions to JDT annotations

This removes run-time reliance on javax.annotation package.

JIRA: YANGTOOLS-907
Change-Id: I9cd870882d8c7edfcc44a50154d0e4148bd4eca5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate common/util to use JDT annotations

This removes run-time reliance on javax.annotation package.

Change-Id: I59193dbe7fcf472fbab29099b685df669c3b301a
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate yang-common to use JDT annotations

This removes run-time reliance on javax.annotation package.

Change-Id: If8870ba4f6aed9b77d65dc129e3fd7112fb01949
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate concepts to use JDT annotations

This removes run-time reliance on javax.annotation package.

Change-Id: Ia4c2115f1160446d781f2d7fa1a7b5a4940a3feb
JIRA: YANGTOOLS-907
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix Java 11 build

This fixes spotbugs/Java11 false positive.

Change-Id: Idb8e648bf35f0c9f73a236dcfc8a8419e800cacb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not import java.annotation

Nullable JSR305 annotations have runtime retention scope, which is
causing javax.annotation being imported by our bundles, which is not
correct as we do not need the annotation. Reduce the use of these
and suppress javax.annotation imports.

JIRA: YANGTOOLS-907
Change-Id: I0e1f384880e148c707037f8afd4b8cb9d2e00e72
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Migrate common/util to use JDT annotations"

This reverts commit f069fc0c4ef1a95ef7ca81a2d83301a48bf18fb8.

Change-Id: Icaa2bf8d664fcacf4cb7ac97bdb796414845d623
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Migrate yang-common to use JDT annotations"

This reverts commit 7286488bb26fbd175297925978c4f75ecee1b70f.

Change-Id: I39afedde0aa2d58b9fe8a809a6f8a7a6de59b770
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Migrate concepts to use JDT annotations"

This reverts commit 99379bf4ef363ff88b1898aac1ad671025407322.

Change-Id: I4c3915be96ba1344ca05933359ed88aa238c8913
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate common/util to use JDT annotations

This removes run-time reliance on javax.annotation package.

JIRA: YANGTOOLS-907
Change-Id: I6665adc378a37b09669de5542b7571a040a5d6f6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>