Do not depend on osgi.annotation

We are using split-out OSGi bundles, make sure our dependencies are
correct.

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

Do not export yang.parser.rfc7950.{namespace,stmt}

These packages are not used anywhere in ODL proper, do not export them
to further isolate implementation internals.

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

Remove yang-parser-antlr

The antlr grammar is an implementation-internal detail which we broke
out to keep spotbugs happy. It turns out we can keep it happy with a
simple suppression.

Move the grammar and eliminate its artifact, making things a bit
smoother.

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

Remove yang-xpath-antlr

The antlr grammar is an implementation-internal detail which we broke
out to keep spotbugs happy. It turns out we can keep it happy with a
simple suppression.

Move the grammar and eliminate its artifact, making things a bit
smoother.

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

Migrate yang-xpath-impl to bnd-parent

bnd-parent is a more modern replacement for bundle-parent, use that.

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

Migrate yang-xpath-antlr to bnd-parent

bnd-parent is a more modern replacement, migrate to use it.

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

Migrate yang-xpath-api to bnd-parent

Use bnd-parent instead of bundle-parent, so we end up with cleaner
integration.

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

Migrate common/util to bnd-parent

We have a better-integrated parent, use that instead of the old felix
plugin. Also add a bit of documentation.

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

Add a uses/refine test

Improve the test model to also check refine target.

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

Add a @return

javadoc does not like us not documenting @return, fix that.

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

Do not subclass XMLUnit

We just want an assert, eliminate an Eclipse warning by using XMLAssert
directly.

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

Hide JsonParserStream.read()

This method seems to be mistakenly exposed, hide it from outside world.

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

Fix YangTextSchemaSource error report

Exception arguments are mismatched, fix that up.

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

Do not enforce augments in unsupported nodes

If our parent uses node is not supported by features, we should not be
trying to enforce the augmentation dependency, as it will not be
present.

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

Disable uses/augment statements of unsupported paths

Uses statements should propagate unsupported-by-features statements
as unsupported-to-build statements.

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

Fix javadoc build

We have merge job failing on javadocs, make sure we repair that.

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

Bump odlparent to 9.0.13

Pick up latest fixes from upstream.

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

Add SoftSchemaSourceCache

GuavaSchemaSourceCache has a rather ugly design, which cannot be easily
scaled. Introduce a simpler SoftSchemaSourceCache as its replacement.

JIRA: YANGTOOLS-1391
Change-Id: I0de9092885ede3efff9d76d48ec14b85a69818b2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 0cd2c12e31e8bdf3463b8282a6c6cc2e6995c140)

Clean up GuavaSchemaSourceCacheTest

This is a rather ancient test, clean it up with try-with-resources
block plus a few other niceties.

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

Simplify GuavaSchemaSourceCache.offer() a bit

We do not have to access the source id multiple times and can lower
cognitive load with an early return.

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

Use representation class for casting

We have an unchecked cast supressed here. Use Class.cast() to get
rid of it.

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

Migrate yang-common to bnd-parent

yang-common is a simple component, migrate it to bind-plugin. Also make
sure package-info.java is correctly named.

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

Migrate concepts to bnd-parent

A straightforward migration to use bnd-maven-plugin.

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

Do not instantiate JsonParser

JsonParser.parse() should not be used, but rather its equivalent static
methods should be invoked. Migrate tests to do so, fixing a few warnings
in the process.

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

Use try-with-resources in TestUtils

Eclipse is pointing out we should be managing resources properly, fix
that.

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

Update XXX to a FIXME

Promote a simple note to a full-blown FIXME, as it seems we should be
able to convert BigDecimal more efficiently than we do today.

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

Do no use Builder concept in RpcResultBuilder

The concept as such will be removed in a future release, make sure we do
not use it internally.

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

Disconnect QNamePredicatedBuilder from Builder

The Builder concept is going away, make sure we do not use it.

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

Disconnect yang-data-util from concepts.Builder

The Builder concept is going away, do not use it during XPath parsing.

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

Do not use concepts.Builder in parser-rfc7950

The Builder concept is going away, do not use it.

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

Disconnect rfc7952-data-util from concepts.Builder

The Builder concept is going away, do not use it in
ImmutableNormalizedMetadata.

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

Bump odlparent to 9.0.11

Pick up latest upgrades from upstream.

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

Disconnect yang-datae-tree-ri from concepts.Builder

concepts.Builder is going away, do not rely on it for our internal
classes.

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

Disconnect DeclaredStatementFormatter from Builder

concepts.Builder is going away, do not use it for
DeclaredStatementFormatter.Builder.

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

Disconnect DataTreeConfiguration.Builder

concepts.Builder is being removed, make sure we do not use it for
DataTreeConfiguration.

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

Deprecate (Checked)Builder for removal

We do not have any users in yangtools and most downstreams, deprecated
these interfaces for removal.

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

Disconnect yang-model-ri from Builder

concepts.Builder is going away, remove its use from yang-model-ri.

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

Disconnect CrossSourceStatementReactor.Builder

concepts.Builder is going away, do not use it in
CrossSourceStatementReactor.

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

Disconnect StatementSupportBundle.Builder from concepts

concepts.Builder is going away, disconnect it from
StatementSupportBundle.

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

Disconnect YangParserConfiguration.Builder from concepts

concepts.Builder is going away, disconnect it from yang-parser-api.

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

Disconnect SchemaContextFactoryConfiguration.Builder

concepts.Builder is going away, make sure we do not use it for
SchemaContextFactoryConfiguration.

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

Disconnect NormalizedNodeBuilder from Builder

concepts.Builder is going away, disconnect it from
NormalizedNodeBuilder.

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

Decouple YangInstanceIdentifierBuilder from Builder

concepts.Builder is going away, make sure we decouple it.

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

Decouple HashCodeBuilder from Builder

We are deprecating Builder concept and HashCodeBuilder does not really
like having to inflate to java.lang.Integer. Ditch the indirection.

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

Remove redundant type arguments

We have Java 11, hence we can use diamond notation for anonymous
subclasses.

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

Deprecate (Checked)Builder

These interfaces are rather overarching, causing callers of build()
method to be opaque. Deprecate them so downstreams can gradually migrate
away.

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

Remove overridden ImmmutableNodes.fromInstanceId()

These methods are geared towards NETCONF filter creation, which is being
solved in a different way -- as these do not work form leaves anyway.
Remove the methods to reduce confusion.

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

Deprecate ImmutableNodes.fromInstanceId() with deepestElement

The two methods allowing an override of the deepest element have only a
single user, which is broken by their semantics. Deprecate them for
removal, so that we flush out anyone else still relying on them.

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

Deprecate SemVer import mode

Semantic version imports are not used by anyone and have a rather
convoluted implementation. Deprecate them along with their supporting
elements.

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

Bump odlparent to 9.0.10

Pick up latest upgrades from upstream.

Change-Id: Idb182da59822f408640750cc6c5cb4d09ea3fc5c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 99828ad40463849ed5cbe26fb8d4d38093c73dc3)

Fix a nullness warning

We are guaranteeing a non-null return, so that takeSnapshot() API
contract is kept.

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

Validate unique statement references

Unique statement's argument needs to be validated against the effective
model, so that we discover when we have mismatches in arguments.

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

Do not mask single exceptions

If we end up with a single exception, do not wrap it in
InferenceException, as we want to correctly identify whether it is a
problem with the source or inference.

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

Populate ietf-restconf operations container

The 'operations' container defined in ietf-restconf module has magic
semantics: it needs to have an empty leaf for every RPC that is
available in the EffectiveModelContext.

WWe are hard-pressed here to make things work with our parser
infrastructure, as we currently do not have hooks to do this kind of
schema-tree specific magic. Luckily there is also a YANG extension in
that same model, so we hijack that to hook onto to the container
inference and modify it just in time and make the magic happen.

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

Remove an unneeded yang-data-spi dependency

We do not need util artifact, do not depend on it.

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

Use NamespaceStmtCtx

We do not need full access here, just namespaces, reflect that.

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

Remove empty lines

We have a few stray empty lines, remove them.

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

Return @NonNull collections in StmtContext

We are always returning non-null contexts, make sure we reflect that
in the return type. This fixes about 20 false positive warnings.

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

Add a @NonNull annotation

Prerequisites are always resolving to a non-null value, make sure
we express that, eliminating a number of warnings.

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

Add SchemaNodeIdentifier-based constructor

We are deprecating SchemaPath, provide a proper constructor allowing
downstreams to use it instead of the SchemaPath-driven one.

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

Correct and/or/div/mod in path expressions

The grammar is creating implicit lexer tokens for and/or/div/mod, which
take precedence overt NCName. Make sure we define these tokens
explicitly and allow them to be an alternative wherever NCName is
valid.

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

Bump odlparent to 9.0.9

Pick up latest fixes from upstream.

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

Simple cleanups

Clean up a few things which got past code review.

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

Trigger onStatementAdded() for replicas

When a statement is introduced through StmtContext.replicaAsChildOf(),
the corresponding statement support should be notified of this fact, so
it can properly do its thing. This makes it consistent with
Mutable.childCopyOf().

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

Deviations should get disabled if target is unsupported

Deviate's InferenceAction should pay attention to unavailable target
nodes, so we disable the deviation.

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

Augments should get disabled if target is unsupported

AugmentInferenceAction should pay attention to unavailable target
nodes, so we disable the augmentation.

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

Require mutatesEffectiveCtxPath() implementation

This method was retrofitted in previous releases, now make it mandatory
to implement.

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

Remove YangModelDependencyInfo.forResource()

This method has been deprecated and users have been migrated. Remove it.

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

Remove DeclaredStatement.statementOrigin()

DeclaredStatements are always declared and thus statementOrigin() should
only be exposed from EffectiveStatements.

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

Remove CommonStmtCtx.origin()

This method should not be used anymore, remove it.

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

Prune StmtContext migration methods

We have a number of migration methods for splitting StmtContext up.
These are no longer used, hence remove them.

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

Remove StmtContextUtils.findFirstSubstatement()

This utility method has an integrated replacement, remove it.

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

Expand perf optimization TODO

Add a few more details on ideas how we can speed up things involving
statement reuse.

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

Deprecate methods throwing UOE

Let's warn on any attempt to directly access an unsupported method.

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

Rework undeclared statement definition

Statements which are created without a corresponding declaration should
not have a DeclaredStatement footprint. Unfortunately our
createEffective() expect a DeclaredStatement to be present.

This patch adds another, explicit, avenue to create an
EffectiveStatement without having its DeclaredStatement (and
rawArgument).

We then use this facility to provide our usual
Undeclared*EffectiveStatements.

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

Filter implicitly-created declared substatements

When we create an implicit parent statement in createDeclaredChild()
we need to exclude that statement from the statements we pass to
createDeclared(). Otherwise the statement would be an 'undeclared'
declared statement, which our downstreams are still working with.

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

Do not expose StmtContext to StatementFactory

We should not be exposing complete inherence state, only the bare
minimum for implementations.

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

Lock down AbstractResumedStatement.substatements access

We are indirecly using mutableDeclaredSubstatements(), and that method
is not final. It is not overridden anywhere, either. So let us make that
strong assumption by making the method final and accessing substatements
directly for their Stream.

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

Expose Iterator from effectiveChildrenToComplete()

We are only ever iterating here, lower the contract exposed through
here.

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

Refactor StatementFactory.createDeclared()

Declared statements need to be built from a different source than
StmtContext.declaredStatements(), as those are following a different
structure.

Rather than creating more confusion in StmtContext, feed a
Stream<DeclaredStatement<?>>. Also drop a FIXME for a follow-up patch
to reduce StmtContext proliferation.

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

Hide AbstractResumedStatement.walkChildren()

We have slight (2-line) duplication due to how gadgets are exposed
to StatementContextWriter.

Introduce AbstractResumedStatement.declarationFinished(), which
performs the combination of walking children and calling endDeclared().

This allows us to completely hide endDeclared() and simplify the code a
bit -- including renaming endDeclared() to finishDeclaration().

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

Move StatementContextWriter.exitStatement()

The logic for entering a declared statement lives in
AbstractResumedStatement, along with the logic to create implicit
parents (i.e. implicit case statements). Also move the code to exit
the declared statement there, so we have all three bits in one place.

Since we have better context for dealing with this, also reshape the
logic, which allows us to move StatementContextBase.endDeclared().

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

Fix a nullness warning

getPhase() is required to be non-null, which we are assuring, but do not
have annotated. Add annotation and move the method, as it is a simple
getter.

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

Move implicit parent skip code

We have an implicit contract between AbstractResumedStatement and
StatementContextWriter during lookup. Since the structure is defined
in AbstractResumedStatement, we should keep the logic together.

Also fix a slight wart, where we would over-allocate implicit parent's
StatementMap.

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

Update StatementMap javadocs

We have two undocumented methods, add javadocs.

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

Cleanup StatementMap a bit

empty() return a non-null instance and we do not need this. qualifier.

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

Add default statementOrigin() implementations

The design of {Model,Declared,Effective}Statement interfaces harks back
to Java 7 days. Refresh it with Java 8's default interface methods,
properly binding them to DeclaredStatement and
EffectiveStatement.getDeclared() methods.

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

Hide StatementContextBase

We have an ugly dependency on StatementContextBase in statement
implementations. These should not care about implementation details,
and in fact they do not, as most of the methods involved are already
present in StmtContext.Mutable.

Expose the rest of the required methods and hide the base class. This
has a nice side effect of removing a significant number of unsafe casts.

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

Delay modifiers during their processing

When we are invoking actions, we are also iterating over them, hence
we have to stop modifiers being added -- otherwise we'll get a CME.
Add the minimal machinery to make this work.

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

Expose addEffectiveSubstatement()

We have partial support for creating effective statements, in the
reactor, but it was not properly structured for all the use cases we
need.

Expose the methods in StmtContext.Mutable and readjust them to allow to
expose statement-agnostic arguments.

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

Deprecate CopyableNode

// FIXME: add jira issue

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

Deprecate AddedByUsesAware for removal

Strenghten deprecation to indicate removal in the next major release.

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

Cache CopyType.values()

We are on critical path accessing values, which end up being cloned
by the implementation. Let's amortize that by remembering the array.

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

Remove TreeNodeFactory

There is no point in having a single-method utility class, just move
the method to TreeNode instead.

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

Make (Mutable)TreeNode a class

A pure interface leads to us having AbstractTreeNode, which is
superfluous. Turn interfaces into abstract classes instead.

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

Eliminate yang.data.spi.tree

We have tests lurking in the wrong artifact (should be yang-data-api)
and the production code is really implementation detail of
yang-data-tree-ri.

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

Split out yang-data-tree-impl

InMemoryDataTree forms a rather large part of yang-data-impl and is
used only for data store implementations. Split it out into its own
component.

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

Split out yang-data-tree-{api,spi}

In order to split out InMemoryDataTree, we need to first split out
its API components. This patch introduces yang-data-tree-{api,spi},
which hold the basic components defining the API and implementation
helper bits.

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

Remove StoreTreeNodes.getChild()

This method is not used anywhere, remove it.

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

Implement ModelStatement.toString()

Make sure we codify hashCode()/equals()/toString() methods in
AbstractModelStatement -- fixing a major usability issue with our
objects and improving consistency.

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