FULL PRODUCT VERSION :
java version "1.5.0"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0-b64)
Java HotSpot(TM) Client VM (build 1.5.0-b64, mixed mode, sharing)


ADDITIONAL OS VERSION INFORMATION :
not OS-related

A DESCRIPTION OF THE PROBLEM :
Careful scrutiny of the source code shows that the gap between the final two components on the line is missing, when testing whether 'maxwidth' has been exceeded.

The extract below has the offending lines commented out, and simple alternatives that correct the problem added.

A more efficient variation is probably to be recommended.


REPRODUCIBILITY :
This bug can be reproduced always.

---------- BEGIN SOURCE ----------
  // extract from java.awt.FlowLayout

  public void layoutContainer(Container target) {
    synchronized (target.getTreeLock()) {
      Insets insets = target.getInsets();
      int maxwidth = target.width - (insets.left + insets.right + /* 2* */hgap);
      int nmembers = target.getComponentCount();
      int x = 0, y = insets.top + vgap;
      int rowh = 0, start = 0;

      boolean ltr = target.getComponentOrientation().isLeftToRight();

      for (int i = 0 ; i < nmembers ; i++) {
Component m = target.getComponent(i);
if (m.visible) {
Dimension d = m.getPreferredSize();
m.setSize(d.width, d.height);

// if ((x == 0) || ((x + d.width) <= maxwidth)) {
if ((x == 0) || ((x + d.width + hGap) <= maxwidth)) {
// if (x > 0) {
// x += hgap;
// }
// x += d.width;
x += d.width + hGap;
rowh = Math.max(rowh, d.height);
} else {
moveComponents(target, insets.left + hgap, y, maxwidth - x, rowh, start, i, ltr);
// x = d.width;
x = d.width + hGap;
y += vgap + rowh;
rowh = d.height;
start = i;
}
}
      }
      moveComponents(target, insets.left + hgap, y, maxwidth - x, rowh, start, nmembers, ltr);
    }
  }

---------- END SOURCE ----------
###@###.### 10/25/04 05:40 GMT