Name: dbT83986 Date: 03/23/99


The implementation of sun.java2d.PeekGraphics.drawString(...)
differs from that of the basic Graphics2D Object. The issue
is that PeekGraphics.drawString(...) does not verify that the
length of the String is greater than zero. Even if the length
of the String is zero it passes it on to java.awt.font.TextLayout,
like this...

public void drawString(String str, int x, int y) {
  drawTextLayout(
    new TextLayout(str, getFont(), getFontRenderContext()), x, y);
}

TextLayout then throws an IllegalArgumentException because it
does not accept Strings of length zero. This issue showed up
while printing the contents of a JTable, which had a few empty
cells.

===========================
REVIEW NOTE 3/23/99 - Similar incident 55551; same user

Similarly, after
 patching the Java runtime tree I found that my fix worked,
almost.
 The same issue is duplicated in
sun.java2d.PathGraphics.drawString(String,int,int). It fails to
check if the length of the String is equal to zero before passing
it on to java.awt.font.TextLayout.

>Workaround
Do this...
public void drawString(String str, int x, int y) {
  if (str.length() != 0) {
    TextLayout layout = new TextLayout(str, getFont(), getFontRenderContext());
    Shape textShape = layout.getOutline(AffineTransform.getTranslateInstance(x, y));
    fill(textShape);
  }
}

I have compiled an run these changes and everything workes fine.
(Review ID: 55542)
======================================================================