On Sat, Oct 02, 2010 at 05:48:07PM -0000, spepp...@apache.org wrote:
> Author: spepping
> Date: Sat Oct  2 17:48:07 2010
> New Revision: 1003845
> URL: http://svn.apache.org/viewvc?rev=1003845&view=rev
> Log:
> Remove unused methods from Hyphenator; this leaves a utility class
> Modified:
>     xmlgraphics/fop/trunk/src/java/org/apache/fop/hyphenation/Hyphenator.java

When I wanted to add configurability for hyphenation pattern file
names, I had to analyse the callers of a lot of Hyphenator methods, to
see if they needed access to the configuration. It turned out that
several methods were never called from within FOP. Even the Hyphenator
constructor is not called from within FOP, so that there never is a
Hyphenator object in FOP. I removed these methods because it
facilitates working on FOP's code, and saves a lot of time. Of course,
there is a remote possibility that these public methods are used by an
external application, but that is so remote as to be beyond my


Simon Pepping
home page: http://www.leverkruid.eu

Reply via email to