[ http://issues.apache.org/struts/browse/SB-49?page=all ]

Antonio Petrelli updated SB-49:
-------------------------------

        Summary: AttributeTag.calculateType is called even if an attribute does 
not need preprocessing  (was: InsertTag.preprocessAttribute is called even if 
an attribute does not need preprocessing)
    Description: 
The AttributeTag.calculateType method is called for every attribute, even those 
that have been already preprocessed.
Therefore there is a (pretty small) performance problem. Currently is a case of 
a null check and a String comparison (plus a call, obiously), but in the future 
it could be more complex.

  was:
The InsertTag.preprocessAttribute is called for every attribute, even those 
that are configured via Tiles configuration files, that are correct after the 
end of the creation.
Therefore there is a (pretty small) performance problem. Currently is a case of 
a null check and a String comparison (plus a call, obiously), but in the future 
it could be more complex.


Updated because InsertTag has been splitted

> AttributeTag.calculateType is called even if an attribute does not need 
> preprocessing
> -------------------------------------------------------------------------------------
>
>                 Key: SB-49
>                 URL: http://issues.apache.org/struts/browse/SB-49
>             Project: Sandbox
>          Issue Type: Improvement
>          Components: Tiles
>    Affects Versions: 2.0
>         Environment: N/A (it's a design issue)
>            Reporter: Antonio Petrelli
>            Priority: Minor
>
> The AttributeTag.calculateType method is called for every attribute, even 
> those that have been already preprocessed.
> Therefore there is a (pretty small) performance problem. Currently is a case 
> of a null check and a String comparison (plus a call, obiously), but in the 
> future it could be more complex.

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

        

Reply via email to