[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components

Nicolas Malin (Jira)

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