[ 
https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14509884#comment-14509884
 ] 

Nicolas Malin commented on OFBIZ-5169:
--------------------------------------

I also agree with you (really it's not a joke)

I started the conversion on OFBIZ-5781 where Deepak Dixit began by groovy file. 
When I kept, I didn't read this thread so I checked what is more used on 
framework.

Now I understand more each detail and I have two choices :
 * revert and restart the conversion (lot's of work and javolution come back)
 * keep like it and realize a second pass with the ArrayList improvement

I really hesitated between both to finally says, ok I break nothing, I didn't 
impacted the framework, so move forward.
And sure, it will be a pleasure for me to review your future patch ;)







> Removing Javolution from framework components
> ---------------------------------------------
>
>                 Key: OFBIZ-5169
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-5169
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Varun Bhansaly
>            Priority: Minor
>         Attachments: framework-FastList-removed.patch
>
>
> a. Remove static instances of Javolution objects.
> b. Remove other uses of Javolution objects.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to