Mark Wielaard wrote:

Hi,

Running the Mauve testsuite (CVS) in preparation of 0.18:

0.17:     30437 PASS, 1216 FAIL
0.18-pre: 31105 PASS, 1059 FAIL

With the following regressions:

+FAIL: 
gnu.testlet.javax.swing.plaf.basic.BasicLookAndFeel.initComponentDefaults: 
PasswordField (number 3)
This one is caused by this patch:

2005-08-16  Lillian Angel  <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>

       * javax/swing/plaf/basic/BasicLookAndFeel.java:
       Changed font for PasswordField because textbox was
       being drawn at an awkward height.
       * javax/swing/text/PasswordView.java
       (getPreferredSpan): Implemented
       (modelToView): likewise.
       (viewToModel): partially implemented.

It's not a serious regression, except to the extent that it is masking a bug in the sizing of text fields.


[Note: 75 others in this suite also failed (and still fail) with 0.17]


Many of those could be fixed relatively easily, since they are just values in a lookup table. But I think it is better to leave the failures there until someone, who is familiar with the relevant component, has checked that the default value is used wherever it should be. For anyone working on the look-and-feel code, it's worth looking through the BasicLookAndFeel.initComponentDefaults test in Mauve since it lists all the default values used (or at least available) in Sun's implementation.

Regards,

Dave


_______________________________________________
Classpath mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath

Reply via email to