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

Taher Alkhateeb commented on OFBIZ-10485:
-----------------------------------------

Third patch implemented in r1836710. Thank you for the feedback Mathieu. I 
think I will lump together a few of the upcoming patches together due to their 
simplicity and brevity, I'll share any questions I might have in here.

I have to mention that I really enjoy two things the most: refactoring and 
declarative / functional coding. We have waaaay to many messy classes in base 
and other framework components that I hope we get the chance to cleanup.

Thank you for your contributions, awesome work! Keep them coming!

 

 

> Refactor MapContext
> -------------------
>
>                 Key: OFBIZ-10485
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10485
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: base
>            Reporter: Mathieu Lirzin
>            Assignee: Taher Alkhateeb
>            Priority: Minor
>             Fix For: Upcoming Branch
>
>         Attachments: OFBIZ-10485_0001-Remove-MapContext-dead-code.patch, 
> OFBIZ-10485_0002-Add-missing-Override-in-MapContext.patch, 
> OFBIZ-10485_0003-Use-the-Deque-interface-in-MapContext.patch, 
> OFBIZ-10485_0004-Rewrite-MapContext-isEmpty.patch, 
> OFBIZ-10485_0005-Rewrite-MapContext-containsKey.patch, 
> OFBIZ-10485_0006-Rewrite-MapContext-keySet.patch, 
> OFBIZ-10485_0007-Remove-MapContext.ListSet-class.patch, 
> OFBIZ-10485_0008-Inline-MapContext-getMapContext.patch, 
> OFBIZ-10485_0009-Rename-stackList-to-maps.patch, 
> OFBIZ-10485_0010-Rework-comments.patch, 
> OFBIZ-10485_0011-Add-entryStream.patch, 
> OFBIZ-10485_0012-Add-withMapContainingKey.patch, 
> OFBIZ-10485_0013-Rewrite-size.patch
>
>
> Following conversation 
> [https://lists.apache.org/thread.htmlf1729ef5eafcf71adfbed0c3ea61dfb73225dff82abc4a57c3de8ed5@%3Cdev.ofbiz.apache.org%3E]
>  on d...@ofbiz.apache.org.
> Here is a first batch of patches to clean things up. Those patches are meant 
> to be applied in order.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to