[ 
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)

Reply via email to