StyleNode objects could be much more lightweight
------------------------------------------------

                 Key: TRINIDAD-633
                 URL: https://issues.apache.org/jira/browse/TRINIDAD-633
             Project: MyFaces Trinidad
          Issue Type: Improvement
            Reporter: Adam Winer


We create a lot of StyleNode objects  ( > 5000 in one test I ran recently), so 
anything that
can be done to make these objects less heavy is a good thing.

A couple things could be done:
- Use singletonList() instead of unmodifiableList() around Arrays.asList() for 
single-element arrays
- Check the incoming arrays not just against null, but also against being 
zero-element arrays,
  and use Collections.emptyList() as necessary there

(SkinStyleSheetParserUtils, in particular, is passing in empty lists for 
includedStyles
and empty sets for inhibitedProperties.)

The former optimization saves 76 bytes every time it happens, the latter 84 
bytes
every time it happens.  This can add up to a good bit of saved memory.

-- 
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