DO NOT REPLY [Bug 25480] New: - [PATCH] Experimental performance improvements.

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25480>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25480

[PATCH] Experimental performance improvements.

           Summary: [PATCH] Experimental performance improvements.
           Product: Fop
           Version: 1.0dev
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: general
        AssignedTo: fop-dev@xml.apache.org
        ReportedBy: bckfnn@worldonline.dk


This (rather large) patch implements different performance improvements to the 
property handling.

- Switched from string based property names to integer enums.
  Most of the lookups then changed from get("font-size") to
  get(P_FONT_SIZE) and compound propery lookup changed to
  get(P_SPACE_AFTER | C_OPTIMUM).
- Cached the non-contextdep default compound properties in the makers.
  This caching is similar to what already existed for the base
  properties.
- Calculated rel2abs and abs2rel in the property makers at codegen time
  to avoid string manipulation at runtime.
- Changed the element lookup in ProperyListBuilder.elementTable from
  using string keys to using integer enums.
- Copy the inherited property values from parent fo into the child
  fo. findProperty() is then no longer recursive.

This patch is not meant to be applied, but is purely for discussion and 
experimentation.