[ 
https://issues.apache.org/jira/browse/XALANJ-2596?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

August Shi updated XALANJ-2596:
-------------------------------
    Attachment: patch.patch

Patch for using singleton default FunctionTable

> Removing unnecessary creation of FunctionTable
> ----------------------------------------------
>
>                 Key: XALANJ-2596
>                 URL: https://issues.apache.org/jira/browse/XALANJ-2596
>             Project: XalanJ2
>          Issue Type: Improvement
>      Security Level: No security risk; visible to anyone(Ordinary problems in 
> Xalan projects.  Anybody can view the issue.) 
>          Components: XPath
>    Affects Versions: 2.7.2
>            Reporter: August Shi
>            Assignee: Steven J. Hathaway
>            Priority: Minor
>         Attachments: patch.patch
>
>
> I found some inefficiencies in creation of org.apache.xpath.FunctionTable 
> instances in the code base. Most of the time, a FunctionTable instance is 
> created and used only to get at the default Functions stored in its static 
> m_functions field. The only time an instance of the FunctionTable is 
> necessary is when custom Functions are being installed (so the places where 
> installFunction() is called), but in the code base there are very few points 
> where this occurs.
> To avoid creation of FunctionTable instances that are of very little use, I 
> introduce a singleton (defined in FunctionTable) that represents the default 
> FunctionTable. In all places that instantiate a new instance of FunctionTable 
> without installing custom Functions, I remove the instantiation and instead 
> use this singleton.
> I will attach a patch file concerning my changes.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@xalan.apache.org
For additional commands, e-mail: dev-h...@xalan.apache.org

Reply via email to