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

http://issues.apache.org/bugzilla/show_bug.cgi?id=28525

[PATCH] New fonts.FontInfo class

           Summary: [PATCH] New fonts.FontInfo class
           Product: Fop
           Version: 1.0dev
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: general
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


Team,

This patch encapsulates the apps.Document font information into a new 
fonts.FontInfo object.  (Also, a few duplicative methods were removed from 
apps.Document, where they can be otherwise obtained by Document.getDriver()--
i.e., getUserAgent(), getLogger().)  apps.Document, in turn, includes this 
object as a member variable.  Other packages and classes that were getting a 
apps.Document in the past will go back* to just getting a FontInfo object, 
where appropriate.

I'll apply in a couple of days if no issues from anyone.  (Please hold off on 
any file changes in the interim affecting these files, if possible.)

Thanks,
Glen

* apps.Document was created by Victor several months back, when he took 
Layout.FontInfo, and incorporated it into this new class (with several new 
methods as well).  Many other classes (e.g., transcoder-related) that just 
needed a FontInfo object started being fed the whole apps.Document object as a 
consequence.  This returns FontInfo as a parameter to these classes, albeit 
from the Fonts package this time around.

Reply via email to