[ http://jira.codehaus.org/browse/DISPL-231?page=all ]
     
fabrizio giustina closed DISPL-231:
-----------------------------------

    Resolution: Fixed

fixed in cvs as suggested. Thanks

> Property files are read for every table generated
> -------------------------------------------------
>
>          Key: DISPL-231
>          URL: http://jira.codehaus.org/browse/DISPL-231
>      Project: DisplayTag
>         Type: Improvement
>   Components: Build/distribution
>     Versions: 1.0
>      Fix For: 1.1
>  Attachments: TableProperties.java
>
>
> Every time a table is generated, the property files are accessed. In 
> TableProperties.java getLocaleResolverInstance(), some code is executed 
> without useing the result. By moving a if-clause to the top of the method, 
> the code won't be executed, and we will get a performance gain.  I have 
> implemented the change in my locale version, at it works fine. I have also 
> attached the new version of TableProperties.java (see line 409).
> Following change has been done:
> class: TableProperties.java
> method: getLocaleResolverInstance()
> Moved if(localeResolver == null){ from line 433 to line 409.
> The localeResolver is null only first time a table is generated, the rest of 
> the tables will use the localeResolver found first time. 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
This SF.Net email is sponsored by the JBoss Inc.
Get Certified Today * Register for a JBoss Training Course
Free Certification Exam for All Training Attendees Through End of 2005
Visit http://www.jboss.com/services/certification for more information
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to