[commit-cp] [bugs #9948] static fields in javax.swing classes that should be final

[Tom Tromey]

This mail is an automated notification from the bugs tracker
 of the project: classpath.

/**************************************************************************/
[bugs #9948] Latest Modifications:

Changes by: 
                Tom Tromey <address@hidden>
'Date: 
                Tue 08/17/2004 at 03:02 (America/Denver)

------------------ Additional Follow-up Comments ----------------------------
I'm checking in the fix for most of this.

Note that KEYBOARD_DIVIDER_MOVE_OFFSET is not defined
as static by the spec.  Looks like a spelling bug in the
JDK to me, as it has the name of a constant.

It isn't initialized as far as I could find.  This remains
a bug in Classpath.  I'm not sure what value it ought to 
have.

I'm leaving this bug report open as a result.







/**************************************************************************/
[bugs #9948] Full Item Snapshot:

URL: <http://savannah.gnu.org/bugs/?func=detailitem&item_id=9948>
Project: classpath
Submitted by: 0
On: Mon 08/09/2004 at 21:24

Category:  None
Severity:  5 - Average
Resolution:  None
Assigned to:  None
Status:  Open
Platform Version:  None


Summary:  static fields in javax.swing classes that should be final

Original Submission:  There are 4 static fields in the javax.swing classes that 
should be declared as final fields:

javax.swing.plaf.basic.BasicSplitPaneUI.KEYBOARD_DIVIDER_MOVE_OFFSET 
javax.swing.plaf.basic.BasicSplitPaneUI.NON_CONTINUOUS_DIVIDER
javax.swing.JDesktopPane.LIVE_DRAG_MODE
javax.swing.JDesktopPane.OUTLINE_DRAG_MODE

This bug was found with our static analysis tool for finding bugs in Java 
programs, FindBugs:
   http://findbugs.sourceforge.net/

Follow-up Comments
------------------


-------------------------------------------------------
Date: Tue 08/17/2004 at 03:02       By: tromey
I'm checking in the fix for most of this.

Note that KEYBOARD_DIVIDER_MOVE_OFFSET is not defined
as static by the spec.  Looks like a spelling bug in the
JDK to me, as it has the name of a constant.

It isn't initialized as far as I could find.  This remains
a bug in Classpath.  I'm not sure what value it ought to 
have.

I'm leaving this bug report open as a result.













For detailed info, follow this link:
<http://savannah.gnu.org/bugs/?func=detailitem&item_id=9948>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/