Bug #: 52513
           Summary: [PATCH] Moving FOUserAgent to the constructor of
           Product: Fop
           Version: 1.1dev
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: general
    Classification: Unclassified

The renderers rely on the FOUserAgent for configuration information, as such, I
believe a cleaner interface would be to put the user-agent in the constructor
of all the Renderers. This change would reduce the mutability of the renderers
and go one step close to a cleaner interface.

All the examples set the user-agent within a few lines of instantiating the
renderer. This gives more legitimacy for moving the user-agent to the

Although this is part of the "public" API, it's not part of the documented API.
It does have a public access modifier, but almost all the FOP classes do. I do
appreciate the arguments for backwards compatibility are valid, however, FOP
can't stay beholden to a dated API solely for reasons of backwards
compatibility. The necessary changes made to any client code should be simple.

Configure bugmail:
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to