upgrade to a maven-checkstyle-plugin for Java8 Sadly all the new versions behave slightly different. That's of course not good but the additional failures are reasonable at least. I went through all the complaints and fixed them.
Project: http://git-wip-us.apache.org/repos/asf/deltaspike/repo Commit: http://git-wip-us.apache.org/repos/asf/deltaspike/commit/91e3a170 Tree: http://git-wip-us.apache.org/repos/asf/deltaspike/tree/91e3a170 Diff: http://git-wip-us.apache.org/repos/asf/deltaspike/diff/91e3a170 Branch: refs/heads/master Commit: 91e3a170edbbf190606147ab4894d2cd59ebfc49 Parents: 00f2ba3 Author: Mark Struberg <strub...@apache.org> Authored: Tue Mar 6 14:51:28 2018 +0100 Committer: Mark Struberg <strub...@apache.org> Committed: Tue Mar 6 14:51:28 2018 +0100 ---------------------------------------------------------------------- .../deltaspike/cdise/tck/ContainerCtrlTckTest.java | 4 ++-- .../core/impl/config/ConfigResolverContext.java | 14 +++++++------- .../core/impl/config/DeltaSpikeConfigInfo.java | 8 ++++---- .../impl/config/EnvironmentPropertyConfigSource.java | 2 +- .../core/impl/config/LocalJndiConfigSource.java | 2 +- .../core/impl/config/TypedResolverImpl.java | 2 +- .../apache/deltaspike/core/impl/jmx/Operation.java | 6 +++--- .../core/impl/message/DefaultMessageContext.java | 4 ++-- .../InjectableGroupedConversationManager.java | 2 +- .../apache/deltaspike/example/viewconfig/Pages.java | 2 +- 10 files changed, 23 insertions(+), 23 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/cdictrl/tck/src/main/java/org/apache/deltaspike/cdise/tck/ContainerCtrlTckTest.java ---------------------------------------------------------------------- diff --git a/deltaspike/cdictrl/tck/src/main/java/org/apache/deltaspike/cdise/tck/ContainerCtrlTckTest.java b/deltaspike/cdictrl/tck/src/main/java/org/apache/deltaspike/cdise/tck/ContainerCtrlTckTest.java index 536508b..f33a199 100644 --- a/deltaspike/cdictrl/tck/src/main/java/org/apache/deltaspike/cdise/tck/ContainerCtrlTckTest.java +++ b/deltaspike/cdictrl/tck/src/main/java/org/apache/deltaspike/cdise/tck/ContainerCtrlTckTest.java @@ -225,11 +225,11 @@ public class ContainerCtrlTckTest cdiContainer.shutdown(); } - @Test @LockedCDIImplementation(versions = { @LockedVersionRange(implementation = CdiImplementation.WELD11, versionRange = "[1.1.14,1.2)"), @LockedVersionRange(implementation = CdiImplementation.WELD20, versionRange = "[2.0.1.Final,2.1)") - }) + }) + @Test public void testShutdownWithInactiveContexts() { CdiContainer cdiContainer = CdiContainerLoader.getCdiContainer(); http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/ConfigResolverContext.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/ConfigResolverContext.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/ConfigResolverContext.java index 4f3997c..bf08982 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/ConfigResolverContext.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/ConfigResolverContext.java @@ -34,39 +34,39 @@ class ConfigResolverContext private boolean evaluateVariables; private boolean propertyAware; - public ConfigResolverContext() + ConfigResolverContext() { } - public ConfigResolverContext setEvaluateVariables(final boolean evaluateVariables) + ConfigResolverContext setEvaluateVariables(final boolean evaluateVariables) { this.evaluateVariables = evaluateVariables; return this; } - public boolean isEvaluateVariables() + boolean isEvaluateVariables() { return evaluateVariables; } - public ConfigResolverContext setProjectStageAware(final boolean projectStageAware) + ConfigResolverContext setProjectStageAware(final boolean projectStageAware) { this.projectStageAware = projectStageAware; return this; } - public boolean isProjectStageAware() + boolean isProjectStageAware() { return projectStageAware; } - public ConfigResolverContext setPropertyAware(final boolean propertyAware) + ConfigResolverContext setPropertyAware(final boolean propertyAware) { this.propertyAware = propertyAware; return this; } - public boolean isPropertyAware() + boolean isPropertyAware() { return propertyAware; } http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/DeltaSpikeConfigInfo.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/DeltaSpikeConfigInfo.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/DeltaSpikeConfigInfo.java index 286806b..6efb1e7 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/DeltaSpikeConfigInfo.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/DeltaSpikeConfigInfo.java @@ -226,24 +226,24 @@ public class DeltaSpikeConfigInfo implements DeltaSpikeConfigInfoMBean private final String value; private final String fromConfigSource; - public ConfigEntry(String key, String value, String fromConfigSource) + ConfigEntry(String key, String value, String fromConfigSource) { this.key = key; this.value = value; this.fromConfigSource = fromConfigSource; } - public String getKey() + String getKey() { return key; } - public String getValue() + String getValue() { return value; } - public String getFromConfigSource() + String getFromConfigSource() { return fromConfigSource; } http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/EnvironmentPropertyConfigSource.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/EnvironmentPropertyConfigSource.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/EnvironmentPropertyConfigSource.java index 5c5ea5d..b1ff8d1 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/EnvironmentPropertyConfigSource.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/EnvironmentPropertyConfigSource.java @@ -30,7 +30,7 @@ package org.apache.deltaspike.core.impl.config; class EnvironmentPropertyConfigSource extends MapConfigSource { - public EnvironmentPropertyConfigSource() + EnvironmentPropertyConfigSource() { super(System.getenv()); initOrdinal(300); http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/LocalJndiConfigSource.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/LocalJndiConfigSource.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/LocalJndiConfigSource.java index bcc4b20..df7aea2 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/LocalJndiConfigSource.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/LocalJndiConfigSource.java @@ -34,7 +34,7 @@ class LocalJndiConfigSource extends BaseConfigSource { private static final String BASE_NAME = "java:comp/env/deltaspike/"; - public LocalJndiConfigSource() + LocalJndiConfigSource() { initOrdinal(200); } http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/TypedResolverImpl.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/TypedResolverImpl.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/TypedResolverImpl.java index df36054..c7d3f85 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/TypedResolverImpl.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/TypedResolverImpl.java @@ -70,7 +70,7 @@ public class TypedResolverImpl<T> implements ConfigResolver.UntypedResolver<T> private T lastValue = null; - public TypedResolverImpl(ConfigImpl config, String propertyName) + TypedResolverImpl(ConfigImpl config, String propertyName) { this.config = config; this.keyOriginal = propertyName; http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/Operation.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/Operation.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/Operation.java index 40920bc..6a9ccab 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/Operation.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/Operation.java @@ -28,18 +28,18 @@ class Operation private final Method operation; private final boolean presentAsTabularIfPossible; - public Operation(final Method operation, final boolean presentAsTabularIfPossible) + Operation(final Method operation, final boolean presentAsTabularIfPossible) { this.operation = operation; this.presentAsTabularIfPossible = presentAsTabularIfPossible; } - public boolean isPresentAsTabularIfPossible() + boolean isPresentAsTabularIfPossible() { return presentAsTabularIfPossible; } - public Method getOperation() + Method getOperation() { return operation; } http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/message/DefaultMessageContext.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/message/DefaultMessageContext.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/message/DefaultMessageContext.java index b776471..44fb945 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/message/DefaultMessageContext.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/message/DefaultMessageContext.java @@ -50,11 +50,11 @@ class DefaultMessageContext implements MessageContext private List<String> messageSources = new ArrayList<String>(); - public DefaultMessageContext() + DefaultMessageContext() { } - public DefaultMessageContext(MessageContext otherMessageContext) + DefaultMessageContext(MessageContext otherMessageContext) { messageInterpolator(otherMessageContext.getMessageInterpolator()); localeResolver(otherMessageContext.getLocaleResolver()); http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/scope/conversation/InjectableGroupedConversationManager.java ---------------------------------------------------------------------- diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/scope/conversation/InjectableGroupedConversationManager.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/scope/conversation/InjectableGroupedConversationManager.java index f88d491..56fcdf7 100644 --- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/scope/conversation/InjectableGroupedConversationManager.java +++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/scope/conversation/InjectableGroupedConversationManager.java @@ -32,7 +32,7 @@ class InjectableGroupedConversationManager implements GroupedConversationManager { private transient volatile GroupedConversationManager conversationManager; - public InjectableGroupedConversationManager(GroupedConversationManager conversationManager) + InjectableGroupedConversationManager(GroupedConversationManager conversationManager) { this.conversationManager = conversationManager; } http://git-wip-us.apache.org/repos/asf/deltaspike/blob/91e3a170/deltaspike/examples/jsf-examples/src/main/java/org/apache/deltaspike/example/viewconfig/Pages.java ---------------------------------------------------------------------- diff --git a/deltaspike/examples/jsf-examples/src/main/java/org/apache/deltaspike/example/viewconfig/Pages.java b/deltaspike/examples/jsf-examples/src/main/java/org/apache/deltaspike/example/viewconfig/Pages.java index 9029958..cac53ed 100644 --- a/deltaspike/examples/jsf-examples/src/main/java/org/apache/deltaspike/example/viewconfig/Pages.java +++ b/deltaspike/examples/jsf-examples/src/main/java/org/apache/deltaspike/example/viewconfig/Pages.java @@ -53,7 +53,7 @@ public interface Pages extends ViewConfig @NavigationParameter.List({ @NavigationParameter(key = "param1", value = "Hey, I come from a navigation parameter"), @NavigationParameter(key = "param2", value = "Hey, It's an interpolated value: #{myBean.aValue}") - }) + }) class NavigationParameterPage implements RedirectedPages { }