[ https://issues.apache.org/jira/browse/TAP5-2739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17738478#comment-17738478 ]
Hudson commented on TAP5-2739: ------------------------------ FAILURE: Integrated in Jenkins build Tapestry » tapestry-trunk-freestyle #182 (See [https://ci-builds.apache.org/job/Tapestry/job/tapestry-trunk-freestyle/182/]) TAP5-2739: tapestry-ioc JDKUtils and DummyLock removed (ben: rev 14864a28b20542e2b57d74e226919a0f66ed3092) * (delete) tapestry-ioc/src/test/groovy/ioc/specs/DummyLockSpec.groovy * (delete) tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/DummyLock.java * (edit) tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/RegistryImpl.java * (delete) tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/util/JDKUtils.java > Remove org.apache.tapestry5.ioc.internal.util.JDKUtils and > org.apache.tapestry5.ioc.internal.util.DummyLock > ----------------------------------------------------------------------------------------------------------- > > Key: TAP5-2739 > URL: https://issues.apache.org/jira/browse/TAP5-2739 > Project: Tapestry 5 > Issue Type: Task > Components: tapestry-ioc > Affects Versions: 5.8.2 > Reporter: Ben Weidig > Assignee: Ben Weidig > Priority: Minor > > The class is a remnant from 5.3->5.4 / Java 5->6 transition. > Right now, it's only used in RegistryImpl to inform the user that JDK 1.5 is > outdated und Java 1.6 is required for Tapestry 5.4+. > In relatity, Java 8 is needed, so it makes no longer sense to have the check. > > The other method of the class, > createLockForThreadLocalCreation()), isnt used at all in Tapestry itself. > The Dummy related org.apache.tapestry5.ioc.internal.util.DummyLock is > deprecated since 5.4 > > Both classes should be removed, because they don't serve any purpose. > Being in an internal package should make the removal not an issue. -- This message was sent by Atlassian Jira (v8.20.10#820010)