Specialize GroupingEffectiveStatement

We can save a field if the argument matches the declared value, which
translates to some tiny savings.

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

Use a dedicated map for schema/data tree namespace

Heap analysis shows that we have roughly 52K SingletonImmutableMaps,
each of which costs 48 bytes. We can store the equivalent information
with a dedicated structure.

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

Specialize ChoiceEffectiveStatement implementations

We can shift default case and argument to a separate class, saving a few
bytes in the usual case.

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

Use EmptyLeafEffectiveStatement in copyLeaf()

If the arguments match, we can reuse empty leaf, as there is no other
state we keep.

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

Use a linear TypedefNamespace

We do not access TypedefNamespace in our code and it usually is empty,
implement it as a linear search map.

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

Specialize ContainerEffectiveStatementImpl

We can improve class layout by omitting the argument if it was unchanged
relative to declaration. Split the implementation into two classes.

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

Defer mutatesEffectiveCtxPath() hookOnto()

When we are resolving statements along schema tree axis we may end up
hitting a statement which is already resolved as unsupported. In that
case we would end up derefencing a known-null action. Defer
initialization until the action is set.

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

Expose conformance type from module

RFC7950 indirectly defines various conformance types. Make sure we
expose that from ModuleEffectiveStatement. The actual value still needs
to be determinated.

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

Move TypDefinition creation

TypedDataSchemaNode has getType(), which we can provide externally as a
utility method. Move the implementation, so we can reconsider caching
this value in future.

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

Optimize LeafListEffectiveStatement layout

We can save a field in case the declared statement's argument matches
the effective argument.

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

Optimize ListEffectiveStatement layout

We can save a field in case the declared statement's argument matches
the effective argument.

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

Adjust LeafEffectiveStatement argument storage

For original leaves the argument matches the declared view, hence
we can save a field.

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

Remove DerivableSchemaNode

DerivableSchemaNode.getOriginal() has a rather large storage cost, which
is not required anywhere in our code base. Remove it.

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

Extract EffectiveStatementState from schema tree statements

SchemaTree statements are the most common statements we copy around. In
order to store them as efficiently as possible. Take advantage of the
new deduplication facilities by extracting significant state.

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

Rename StmtContext.Mutable.setIsSupportedToBuildEffective()

This method is a mouthful and is really used to set the flag to false,
rename it and adjust callers.

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

Intern statements along copy axis

When we are inferring statements through grouping/uses, we end up adding
a few bits to record the copy history and effective status/config. This
process does not feed back anything back to the original declaration
site.

This leads to a situation when we can have a number of inferred
statements which are exactly equal, yet each is its own java object --
hence unnecessarily using heap for duplicate objects.

We can improve this by having a deduplication map at the site of
original declaration and updating/consulting it when we find we have an
otherwise-unmodified statement.

This patch adds the prerequisite constructs to yang-parser-spi and
the smarts to perform this deduplication to yang-parser-reactor.

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

Do not retain single-entry RangeSets

ImmutableRangeSet with a single entry is a tad inefficient: we retain
the set and also a singleton list. Rather than doing that, store only
the single range -- reducing the number of objects we retain.

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

Expose substatement lists

DeclaredStatement.declaredSubstatements() and
EffectiveStatement.effectiveSubstatements() should be exposing a List
instead of a Collection to make things easier on users.

Also update yang-model-{spi,ri} to promise an ImmutableList, as that
is what they provide.

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

Intern PatternExpressions

Some of the patterns out there are widely duplicated, expecially in
Junos models. Since a PatternExpression is an Immutable object, it can
easily be interned and thus shared across reactors, eliminating ~33K
duplicate objects.

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

Decouple choice/case statements

Use StatementSupportNamespace to perform on-demand lookup of
CaseStatementSupport instead of hard-wiring it at instantiation time.

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

Add StatementSupportNamespace

Expose StatementSupports registered for a particular root from a new
namespace, StatementSupportNamespace. SourceSpecificContext already has
all the knowledge and wiring required to make this work, so it is a
rather straightforward exercise.

This immediately allows us to loosen couplig betwen action/rpc and
input/output supports. Unfortunately choice/case coupling still remains
as we do not get a reference to the statement context in execution path.

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

Remove ModulesDeviatedByModules.SupportedModules

We require only a non-null key, there is no point in defining an
enumeration for that -- just use yang.common.Empty, just as we do in
other places.

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

Remove SupportedFeaturesNamespace.SupportedFeatures

We require only a non-null key, there is no point in defining an
enumeration for that -- just use yang.common.Empty, just as we do in
other places.

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

Reflect supported features in effective model

We currently do not capture the set of supported features in
EffectiveModelContext aside from applying their effects. Improve this
by not exposing unsupported features in the effective model, so that
users can understand the set of supported features by simply looking
for all FeatureEffectiveStatements.

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

Remove Empty.getInstance()

Remove deprecated method. Also promote Empty to stable status by
removing @Beta annotation.

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

Add QNameModule documentation

QNameModule is a widely-used class, make sure it is documented.

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

Add Empty.value()

Introduce a more friendly name for returning the empty instance. The new
method is 'value()' and 'getInstance()' is deprecated.

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

Correct bundle-parent's coordinates

We are inheriting SCM section from odlparent, make sure we add an
override.

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

Use Map.entry() in ModuleNamespaceContext

This is simpler and does not need identity in future.

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

Fix mandatory enforcer failure on augmented nodes

Changes the way mandatory nodes are extracted from the schema.
Previously all mandatory nodes were approached as direct children of
the parent node. Even if they came from an augmentation which caused
"missing mandatory descendant" exceptions. This patch preserves both
options of building data nodes containing augmented mandatory nodes:
- setting the mandatory node as a direct child of the parent node
- setting the mandatory node as an AugmentationNode to the parent node

However the former option seems to be incorrect and might be removed
in the future major release.

This patch also covers the case where the mandatory leaf is deeper in
the augmented subtree.

JIRA: YANGTOOLS-1276
Change-Id: Ic5cd1950b935720bf56f52cfbc24c0f9e776a474
Signed-off-by: Tibor Král <tibor.kral@pantheon.tech>

Revert "Revert "Fix mandatory enforcer failure on augmented nodes""

This reverts commit 7c0832aec2141aa95dd2b6a2d2dcc5501410c2ad. The correct
solution is built on top of this change.

JIRA: YANGTOOLS-1276
Change-Id: Id560049304cf37d62050fbf5ad59c995d11939a5
Signed-off-by: Tibor Král <tibor.kral@pantheon.tech>

Revert "Move SchemaNodeIdentifier to yang-common"

This reverts commit e8049b7d125af0f60c608623683a31d0f366a462, as it is
not the right thing to do. The problem is that it increases exposure of
SchemaNodeIdentifier to components (Binding, for example), where it is
causing cognitive pollution.

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

Require java.compiler in yang-data-api

We are using immutables.org and allow @Generated to be emitted, hence
we need to ensure it is visible to users.

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

Note future YangErrorInfo evolution

We need to evolve this contract based on experience gained during
implementation.

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

Use VarHandle for toStringCache

We are using VarHandle for hashCode, there let's do the same for
toStringCache.

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

Move SchemaNodeIdentifier to yang-common

SchemaNodeIdentifier has not further ties to yang-model-api and maps out
a well-defined core YANG construct. Move it to yang-common.

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

Allow rebinding the prefix of an UnresolvedQName

There are callers who are dealing with prefix being optional, possibly
added (or changed) in a later context. Add the ability to do that
without re-validating the local name.

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

Remove SchemaNodeIdentifier.asSchemaPath()

Disconnect SchemaNodeIdentifier from SchemaPath. The replacement
lives as SchemaPath.of() family of methods.

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

Add SchemaPath.of(SchemaNodeIdentifier)

SchemaNodeIdentifier depends on SchemaPath, which is a bad idea from
flexibility perspective. Since the methods are deprecated, add their
non-cached counterparts in SchemaPath.

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

Improve UnresolvedQNameReferent

We now have UnresolvedQName, use it to mark that the required return
type. Since the two specializations now clash on return type, the two
interfaces cannot be implemented even without QNameReferentBehavior
magic.

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

Fix YANG export with duplicate imports

When we have two submodules not agreeing on the prefix used to import a
module, we get a nasty splat from StatementPrefixResolver. Fix this by
detecting when this is happens and skip to full resolution magic.

JIRA: YANGTOOLS-1313
Change-Id: I99cc3ebc2f560590d05b51287d08429aaf7d6582
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Dominik Vrbovsky <dominik.vrbovsky@pantheon.tech>

Bump odlparent to 9.0.8

Adopt latest fixes from upstream.

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

Bump odlparent to 9.0.7

Adopt latest fixes from upstream.

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

Generate an execution report from yang-maven-plugin-it

Adjust our wiring to pass down failsafe argLine to our individual
maven invocations. This ends up producing correct execution reports.

The second part end up merging these partial reports into a single
execution report.

The third part is to unpack yang-maven-plugin and its dependencies,
which means classes can be looked up and jacoco thinks this is coming
from us. Yeah, not nice, but works.

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

Bump maven-verifier to 1.7.2

This modernizes the components used and removes the need for an
exclusion.

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

Update ANTLR grammar documentation

We have a mis-reference and its parser section needs point out that
the end result is not a complete YANG structure as per ABNF, but rather
the equivalent of lexer stream, which is interpreted in Java code.

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

Deprecate EffectiveStmtCtx.Current.original()

These methods are used to pass information to getOriginal(Definition)
methods, which themselves are deprecated for removal. Mirror that
decision so we do not end up exposing things we do not need.

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

Deprecate AddedByUsesAware

isAddedByUses() is used only by mdsal's binding components, which should
be able to get by without this guidance. Deprecate the interface along
with its support bits.

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

Add more deprecations around SchemaPath

We have a few constructs which provide bridging to SchemaPath, which
are now triggering warnings. Deprecate them, so that we suppress
warnings and give users some time to adjust.

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

Deprecate DerivableSchemaNode

This interface exposes a single method, which in turn is only used
by mdsal-binding-generator. That component should be able to work
without this method, hence we deprecate it with the intent to remove
as soon as practical.

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

Deprecate AugmentationSchemaNode.getOriginalDefinition()

This method is on its way out, mark them as deprecated, so downstreams
are aware of that.

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

Improve AbstractYangTest a bit

Do not declare Exception as thrown but turn it into an assertion.

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

Cleanup code duplication

We have a conditional parent sweep block repeated twice, add a private
method to host it.

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

Fix StatementContextBase.childCopyOf()

Most of our StmtContext implementations are derived from
StatementContextBase, but notably ReplicaStatementContext is not -- and
we do not handle it during copy operations.

Update the dispatch code to short circuit to replicaAsChildOf(), which
takes care of the details.

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

Fix yang-export warnings

We have a number of tests using deprecated methods, migrate them
to silence warnings. Also schedule those methods for removal.

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

Validate cross-revision imports and includes

We are not performing no YANG version validation during include/import
linkage. RFC7950 section 12 specifically forbids a number of cases which
are invalid. Make sure we are enforcing them.

This flushes out a few violations in our test suite, which is also
updated.

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

Fix a mis-import

We do not want to be importing checker.qual.K here, and eclipse warns
about shadowing. Fix that.

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

Cleanup Bug6870Test

Use AbstractYangTest to reduce verbosity and duplication.

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

Cleanup Bug6897Test

Migrate to AbstractYangTest and perform general housekeeping.

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

Add AbstractYangTest

We have a number of common assertion patterns, let's start creating a
library.

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

Move printing utilities out of StmtTestUtils

These are used by a single test, move them there.

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

Remove more TestUtils classes

We have some duplication still and some assert code which should be with
its users in AugmentTest.

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

Cleanup TestUtils.parseYangSources()

We are being needlessly indirect here, as we are always loading
resources. Clean that up.

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

Remove unused TestUtils methods

We have some code duplication and unused code paths, clean that up.

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

Improve {Bug394,TwoRevisions}Test

Improved loading and layout.

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

Remove TestUtils.parseYangSources

We have loadModules() used more widely, eliminate code duplication. Also
cleanup callers with better assertions.

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

Add AbstractModelTest

We have a number of tests which are sharing same EffectiveModelContext,
conceptrate loading and constants and subclass the test. Results in
faster execution and quite cleaner tests.

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

Cleanup ParsingExtensionValueTest

Simpler loading and better asserts.

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

Speed up YangParserTest

We have multiple tests sharing the same context, use @BeforeClass
and cleanup loading.

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

Speed up YangParserSimpleTest

Use new loading to load a constant context, shared between the test
cases.

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

Improve UsesAugmentTest

We have two tests sharing the reactor, use beforeClass() build it only
once and use better loading method to do that.

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

Cleanup SubstatementValidatorTest

We have a ton of missing asserts and try/catch checking, clean all of
that up.

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

Use java.nio.file.Path in YangTextFileSchemaSource

Switching to NIO seems to be a better strategy, as we ned up using it to
access content anyway.

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

Remove concepts.Path

We have a replacement in HierarchicalIdentifier. Users should have
adapted since last release, remove Path now.

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

Introduce HierarchicalIdentifier to replace Path

Path is always combined with Identifier, and it really is an extension
of Identifier contract. Add a new interface to capture that contract
and deprecate Path, so we get out of java.nio.file.Path's way.

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

Inline Bug7480Test model assembly

We have a few utilities which are noize in StmtTestUtils, move them to
heir sole user. This makes it wee bit more obvious what is going on.
This creates a very useful litte utility in TestUtils.

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

Cleanup Bug6874Test

We are using ugly asserts and try-catch blocks, clean that up.

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

Remove a superfluous line

Empty line, remove it.

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

Add a few more utilities to EffectiveModelContext

Locating modules by name/namespace is useful, provide a simple bridge on
top of SchemaContext.findModules().

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

Improve SchemaInferenceStack error reporting

Include a description of parent's identity when we fail to locate
its child. This makes it easier to understand what is going on in most
failure situations.

The message now ends with something like
- not present in schema parent (namespace)parentName
- not present in grouping (namespace)groupingName
- not present in module (namespace)moduleName
or similar.

Also expand the test suite to cover negative scenarios around
enterChoice().

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

Deprecate IllegalArgumentCodec for removal

As previously noted, this interface encourages bad patterns. All
downstream users are better served with their domain-specific
interfaces. Deprecate it with the intent to remove.

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

Flatten IllegalArgumentCodec class hierarchy

IllegalArgumentCodec is the only specialization which is seeing a
wide use. Integrate all its superinterfaces and superclasses of
AbstractIllegalArgumentCodec into these two constructs.

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

Introduce UnresolvedQName

Our AbstractQName hierarchy has either a resolved QName, or two
unresolved {Qualified,Unqualified}QNames. The quality of being resolved
vs. unresolved is not modeled.

Introduce UnresolvedQName as the only AbstractQName alternative to
QName, so that it can be used to model this distinction. This allows,
for example, yang-xpath-api to be specified better.

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

Remove util.Immutables

This class has been deprecated for removal in a previous release, remove
it now.

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

Deprecate util.Immutables for removal

This class is rather incompete in its mission and it is not used
anywhere. Deprecate it for removal.

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

Remove ProductAwareBuilder

This interface has been deprecated in 7.0.9, remove it in version 8.0.0.

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

Deprecate ProductAwareBuilder

This interface is not used anywhere, deprecate it for removal.

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

Remove ClassBasedPropertyBuilder

This interface has been deprecated in 7.0.9, remove it in 8.0.0.

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

Deprecate ClassBasedPropertyBuilder

This interface is not used anywhere, deprecate it for removal.

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

Mark MutationBehaviour evolution

We want to nail seal and potentially hide this interface.

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

Publish identifier matches

Our private matches are useful for other parsers which need to handle
YANG's identifier ABNF production. Document and publish them from
YangNames.

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

Use yang.common.Decimal64 for decimal64

We now have a dedicated class for carrying decimal64 values, use that
instead of BigInteger.

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

Fix Decima64.valueOf(String)

When we have a maximum-length string we end up reporting running out of
fraction limit. Fix this by correctly accounting for the period.

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

Fix CanonicalValueViolation.getMessage()

We should not be looking at the tag but rather at message. This fixes
anonymous exceptions being thrown from Decimal64's value support.

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

Fix DerivableSchemaNode.getOriginal()

We should be returning a type capture here, fix that up, which prompts
us having to retrofit the interface properly into concrete SchemaNodes
-- making it obvious where we have some overlap.

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

Use ErrorTag for RpcError.getTag()

We have a dedicated class (and constants) to cover error-tag, use it
instead of plain String.

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

Remove deprecated NETCONF error interfaces

RpcError.Error{Severity,Type} as well as YangError have their
replacements, hence remove them.

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

Fix EmptyLeafListEffectiveStatement.getOriginal()

We should not be referencing LeafSchemaNode, but LeafListSchemaNode.

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

Fix YangErrorInfo.value

YangErrorInfo is transporting a child of a container, which means it
should not be a plain NormalizedNode, but DataContainerChild instead.

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

Document PrettyTreeIndent.INDENT_STRINGS_SIZE

Explain what we are doing here, so that our future selves can properly
update the constant in the future.

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