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 20:09:44 UTC
[sling-org-apache-sling-caconfig-impl] branch master updated:
cosmetic: remove obsolete @SuppressWarnings tags
This is an automated email from the ASF dual-hosted git repository.
sseifert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-caconfig-impl.git
The following commit(s) were added to refs/heads/master by this push:
new 8335673 cosmetic: remove obsolete @SuppressWarnings tags
8335673 is described below
commit 833567342df318821f808e48d3a7c4bed4ec0adb
Author: sseifert <ss...@pro-vision.de>
AuthorDate: Tue May 21 22:09:23 2019 +0200
cosmetic: remove obsolete @SuppressWarnings tags
---
.../apache/sling/caconfig/impl/ConfigurationBindingsValueProvider.java | 2 +-
.../apache/sling/caconfig/management/impl/ConfigurationDataImpl.java | 1 -
.../java/org/apache/sling/caconfig/management/impl/ValueInfoImpl.java | 1 -
.../java/org/apache/sling/caconfig/impl/ConfigurationProxyTest.java | 1 -
4 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBindingsValueProvider.java b/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBindingsValueProvider.java
index 8cc9fbe..37e87be 100644
--- a/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBindingsValueProvider.java
+++ b/src/main/java/org/apache/sling/caconfig/impl/ConfigurationBindingsValueProvider.java
@@ -72,7 +72,7 @@ public class ConfigurationBindingsValueProvider implements BindingsValuesProvide
private boolean enabled;
@Override
- @SuppressWarnings({ "unused", "null" })
+ @SuppressWarnings("unused")
public void addBindings(Bindings bindings) {
if (!enabled || !bindings.containsKey(SlingBindings.REQUEST)) {
return;
diff --git a/src/main/java/org/apache/sling/caconfig/management/impl/ConfigurationDataImpl.java b/src/main/java/org/apache/sling/caconfig/management/impl/ConfigurationDataImpl.java
index fb7e24b..54553fb 100644
--- a/src/main/java/org/apache/sling/caconfig/management/impl/ConfigurationDataImpl.java
+++ b/src/main/java/org/apache/sling/caconfig/management/impl/ConfigurationDataImpl.java
@@ -261,7 +261,6 @@ final class ConfigurationDataImpl implements ConfigurationData {
}
@Override
- @SuppressWarnings("null")
public boolean isInherited() {
// detect if the whole config or config item was inherited
if (resolvedConfigurationResource != null && resolvedConfigurationResource.getPath() != null) {
diff --git a/src/main/java/org/apache/sling/caconfig/management/impl/ValueInfoImpl.java b/src/main/java/org/apache/sling/caconfig/management/impl/ValueInfoImpl.java
index bde459c..4860080 100644
--- a/src/main/java/org/apache/sling/caconfig/management/impl/ValueInfoImpl.java
+++ b/src/main/java/org/apache/sling/caconfig/management/impl/ValueInfoImpl.java
@@ -109,7 +109,6 @@ final class ValueInfoImpl<T> implements ValueInfo<T> {
}
@Override
- @SuppressWarnings("null")
public boolean isInherited() {
if (isDefault() || effectiveValue == null) {
return false;
diff --git a/src/test/java/org/apache/sling/caconfig/impl/ConfigurationProxyTest.java b/src/test/java/org/apache/sling/caconfig/impl/ConfigurationProxyTest.java
index 2280a79..774b341 100644
--- a/src/test/java/org/apache/sling/caconfig/impl/ConfigurationProxyTest.java
+++ b/src/test/java/org/apache/sling/caconfig/impl/ConfigurationProxyTest.java
@@ -252,7 +252,6 @@ public class ConfigurationProxyTest {
return ConfigurationProxy.get(child, clazz, childResolver(child));
}
@Override
- @SuppressWarnings("null")
public <T> Collection<T> getChildren(String configName, Class<T> clazz) {
List<T> collection = new ArrayList<>();
Resource childParent = resource!=null ? resource.getChild(configName) : null;