[java-metadata-aggregator] branch main updated: Nullability annotations
Ian Young
ian at iay.org.uk
Tue Oct 18 11:00:26 UTC 2022
This is an automated email from the git hooks/post-receive script.
iay pushed a commit to branch main
in repository java-metadata-aggregator.
View the commit online:
http://git.shibboleth.net/view/?p=java-metadata-aggregator.git;a=commit;h=5dbb7484ab9097acf0fe1ec26ceec6e73f3cad82
The following commit(s) were added to refs/heads/main by this push:
new 5dbb748 Nullability annotations
5dbb748 is described below
commit 5dbb7484ab9097acf0fe1ec26ceec6e73f3cad82
Author: Ian Young <ian at iay.org.uk>
AuthorDate: Tue Oct 18 12:00:22 2022 +0100
Nullability annotations
---
.../java/net/shibboleth/metadata/dom/AbstractDOMTraversalStage.java | 5 +++--
.../net/shibboleth/metadata/dom/AbstractElementValidationStage.java | 2 ++
.../net/shibboleth/metadata/dom/AbstractElementVisitingStage.java | 1 +
.../java/net/shibboleth/metadata/dom/SimpleDOMTraversalContext.java | 2 ++
.../net/shibboleth/metadata/dom/StringElementValidationStage.java | 1 +
.../net/shibboleth/metadata/pipeline/AbstractIteratingStage.java | 2 +-
6 files changed, 10 insertions(+), 3 deletions(-)
diff --git a/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractDOMTraversalStage.java b/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractDOMTraversalStage.java
index a87a3db..7c44beb 100644
--- a/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractDOMTraversalStage.java
+++ b/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractDOMTraversalStage.java
@@ -59,6 +59,7 @@ public abstract class AbstractDOMTraversalStage<C extends DOMTraversalContext>
*
* @return an appropriate context
*/
+ @Nonnull
protected abstract C buildContext(@Nonnull final Item<Element> item);
/**
@@ -95,7 +96,7 @@ public abstract class AbstractDOMTraversalStage<C extends DOMTraversalContext>
private void traverse(@Nonnull final Element element, @Nonnull final C context)
throws StageProcessingException {
final List<Element> children = ElementSupport.getChildElements(element);
- for (final Element child : children) {
+ for (@Nonnull final Element child : children) {
traverse(child, context);
}
if (applicable(element, context)) {
@@ -104,7 +105,7 @@ public abstract class AbstractDOMTraversalStage<C extends DOMTraversalContext>
}
@Override
- protected void doExecute(final Item<Element> item) throws StageProcessingException {
+ protected void doExecute(@Nonnull final Item<Element> item) throws StageProcessingException {
final Element docElement = item.unwrap();
final C context = buildContext(item);
traverse(docElement, context);
diff --git a/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementValidationStage.java b/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementValidationStage.java
index 0f4e797..fa77de6 100644
--- a/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementValidationStage.java
+++ b/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementValidationStage.java
@@ -92,6 +92,7 @@ public abstract class AbstractElementValidationStage<T> extends AbstractDOMValid
* @param element {@link Element} being validated
* @return converted value
*/
+ @Nonnull
protected abstract T convert(@Nonnull final Element element);
@Override
@@ -101,6 +102,7 @@ public abstract class AbstractElementValidationStage<T> extends AbstractDOMValid
}
@Override
+ @Nonnull
protected DOMTraversalContext buildContext(@Nonnull final Item<Element> item) {
return new SimpleDOMTraversalContext(item);
}
diff --git a/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementVisitingStage.java b/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementVisitingStage.java
index 992b9e4..5c9fbe8 100644
--- a/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementVisitingStage.java
+++ b/mda-framework/src/main/java/net/shibboleth/metadata/dom/AbstractElementVisitingStage.java
@@ -86,6 +86,7 @@ public abstract class AbstractElementVisitingStage extends AbstractDOMTraversalS
}
@Override
+ @Nonnull
protected DOMTraversalContext buildContext(@Nonnull final Item<Element> item) {
return new SimpleDOMTraversalContext(item);
}
diff --git a/mda-framework/src/main/java/net/shibboleth/metadata/dom/SimpleDOMTraversalContext.java b/mda-framework/src/main/java/net/shibboleth/metadata/dom/SimpleDOMTraversalContext.java
index efedfed..516f426 100644
--- a/mda-framework/src/main/java/net/shibboleth/metadata/dom/SimpleDOMTraversalContext.java
+++ b/mda-framework/src/main/java/net/shibboleth/metadata/dom/SimpleDOMTraversalContext.java
@@ -29,6 +29,7 @@ import net.shibboleth.metadata.Item;
public class SimpleDOMTraversalContext implements DOMTraversalContext {
/** The {@link Item} this traversal is being performed on. */
+ @Nonnull
private final Item<Element> item;
/**
@@ -41,6 +42,7 @@ public class SimpleDOMTraversalContext implements DOMTraversalContext {
}
@Override
+ @Nonnull
public final Item<Element> getItem() {
return item;
}
diff --git a/mda-framework/src/main/java/net/shibboleth/metadata/dom/StringElementValidationStage.java b/mda-framework/src/main/java/net/shibboleth/metadata/dom/StringElementValidationStage.java
index 013c1ae..a4593be 100644
--- a/mda-framework/src/main/java/net/shibboleth/metadata/dom/StringElementValidationStage.java
+++ b/mda-framework/src/main/java/net/shibboleth/metadata/dom/StringElementValidationStage.java
@@ -31,6 +31,7 @@ import net.shibboleth.metadata.pipeline.Stage;
public class StringElementValidationStage extends AbstractElementValidationStage<String> {
@Override
+ @Nonnull
protected String convert(@Nonnull final Element element) {
return element.getTextContent();
}
diff --git a/mda-framework/src/main/java/net/shibboleth/metadata/pipeline/AbstractIteratingStage.java b/mda-framework/src/main/java/net/shibboleth/metadata/pipeline/AbstractIteratingStage.java
index dba4b2b..6640160 100644
--- a/mda-framework/src/main/java/net/shibboleth/metadata/pipeline/AbstractIteratingStage.java
+++ b/mda-framework/src/main/java/net/shibboleth/metadata/pipeline/AbstractIteratingStage.java
@@ -44,7 +44,7 @@ public abstract class AbstractIteratingStage<T> extends AbstractStage<T> {
@Override
protected void doExecute(@Nonnull @NonnullElements final List<Item<T>> items)
throws StageProcessingException {
- for (final Item<T> item : items) {
+ for (@Nonnull final Item<T> item : items) {
doExecute(item);
}
}
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
More information about the commits
mailing list