On 02/10/10 19:37, Simon Pepping wrote: > 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 > horizon.
... and I couldn’t agree more. Thanks for this clean-up work :-) Vincent