You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ss...@apache.org on 2019/05/21 16:45:05 UTC
[sling-org-apache-sling-caconfig-impl] branch
feature/SLING-8272-node-existence-check created (now 0eb9489)
This is an automated email from the ASF dual-hosted git repository.
sseifert pushed a change to branch feature/SLING-8272-node-existence-check
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-caconfig-impl.git.
at 0eb9489 SLING-8272 add @NonNull annotations as in interface to avoid code warnings
This branch includes the following new commits:
new 0eb9489 SLING-8272 add @NonNull annotations as in interface to avoid code warnings
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
[sling-org-apache-sling-caconfig-impl] 01/01: SLING-8272 add
@NonNull annotations as in interface to avoid code warnings
Posted by ss...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
sseifert pushed a commit to branch feature/SLING-8272-node-existence-check
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-caconfig-impl.git
commit 0eb94897b49441588e5761834b9f760039c0abcd
Author: sseifert <ss...@pro-vision.de>
AuthorDate: Tue May 21 18:38:15 2019 +0200
SLING-8272 add @NonNull annotations as in interface to avoid code warnings
---
.../org/apache/sling/caconfig/impl/ConfigurationBuilderImpl.java | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBuilderImpl.java b/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBuilderImpl.java
index 11fc3c2..906bf09 100644
--- a/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBuilderImpl.java
+++ b/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBuilderImpl.java
@@ -397,18 +397,19 @@ class ConfigurationBuilderImpl implements ConfigurationBuilder {
}
}
- // --- Config Node Existence Check Support ---
+ // --- Config Node Existence Check Support ---
+
@Override
- public <T> boolean has(Class<T> clazz) {
+ public <T> boolean has(@NotNull Class<T> clazz) {
final String name = getConfigurationNameForAnnotationClass(clazz);
if (log.isDebugEnabled()) {
log.debug("Check configuration for context path {}, name '{}', class {}", contentResource.getPath(), name, clazz.getName());
}
return checkIfConfigNodeExists(name);
}
-
+
@Override
- public boolean has(String configName) {
+ public boolean has(@NotNull String configName) {
if (log.isDebugEnabled()) {
log.debug("Check configuration for context path {}, configuration name '{}' ", contentResource.getPath(), configName);
}