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

Adrian Crum commented on OFBIZ-1924:
------------------------------------

Thanks for the advice David. I checked out the cache contents in WebTools and 
saw cache keys like:

0
1
Y
N
true
false

and it just seemed silly to cache everything. I'd like to commit it as-is. If 
someone feels strongly otherwise, it can be changed by removing a few lines.

We will still save a lot of memory because empty expressions are sharing a 
common class instance.


> Sandbox: Improved FlexibleStringExpander class
> ----------------------------------------------
>
>                 Key: OFBIZ-1924
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1924
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>            Reporter: Adrian Crum
>            Priority: Minor
>         Attachments: FlexibleStringExpander.patch, 
> FlexibleStringExpander.patch, FlexibleStringExpander_Complete_Refactor.patch
>
>
> As discussed on the dev mailing list, implement an improved 
> FlexibleStringExpander that uses a factory method to create class instances.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to