You are viewing a plain text version of this content. The canonical link for it is here.

Posted to fop-dev@xmlgraphics.apache.org by "J.Pietschmann" <j3...@yahoo.de> on 2003/10/07 21:29:26 UTC

Re: cvs commit: xml-fop/src/java/org/apache/fop/util CharUtilities.java

vmote@apache.org wrote:
>    public class CharUtilities {
...
>   +    public static int getWordWidth(String word, Font fs) {

That's ugly. Shouldn't this be a method of Font?

J.Pietschmann

RE: cvs commit: xml-fop/src/java/org/apache/fop/util CharUtilities.java

Posted by Victor Mote <vi...@outfitr.com>.

J.Pietschmann wrote:

> vmote@apache.org wrote:
> >    public class CharUtilities {
> ...
> >   +    public static int getWordWidth(String word, Font fs) {
>
> That's ugly. Shouldn't this be a method of Font?

Although I don't recognize it as ugly, I have no objection to your proposal.
I was making it parallel with CharUtilities.getCharWidth(). Both of these
methods were in Font (or FontState) in maintenance branch, so I assume that
somebody moved getCharWidth. getWordWidth is needed for the Pioneer LS, and
it may go away at some point.

Victor Mote