[commit-cp] [bugs #11368] JLabel.setDisplayedMnemonic crashes if given char is not found

[anonymous]

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

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

Changes by: Anonymous user
Date:  
                Wed 12/22/2004 at 12:19

------------------ Additional Follow-up Comments ----------------------------
Tested with kaffe-cvs and small test program - all seems ok.

This bug can be closed.






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

URL: <http://savannah.gnu.org/bugs/?func=detailitem&item_id=11368>
Project: classpath
Submitted by: 0
On: Tue 12/21/2004 at 17:30

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


Summary:  JLabel.setDisplayedMnemonic crashes if given char is not found

Original Submission:  
Caused by: java.lang.StringIndexOutOfBoundsException: String index out of 
range: -1
   at java.lang.String.charAt (String.java:476)
   at javax.swing.JLabel.setDisplayedMnemonicIndex (JLabel.java:418)
   at javax.swing.JLabel.setDisplayedMnemonic (JLabel.java:375)
   at javax.swing.JLabel.setDisplayedMnemonic (JLabel.java:388)

Sun's javax does nothing on that case - also in Suns
JavaDoc there is no exception specified.

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


-------------------------------------------------------
Date: Wed 12/22/2004 at 12:19       By: 0 <None>
Tested with kaffe-cvs and small test program - all seems ok.

This bug can be closed.

-------------------------------------------------------
Date: Tue 12/21/2004 at 22:41       By: Michael Koch <mkoch>
I forgot to commit a little change. Its in now.

-------------------------------------------------------
Date: Tue 12/21/2004 at 21:52       By: 0 <None>
I can't test it - my project compiles but does not run yet:
now IllegalThreadStateException in GtkWindowPeer.create - i'll try to report 
later.

Looking at JLabel.java, seems its not right yet - Sun's code
finds mnemonic 'A' from String "aaa" - that means that
text should be also put to upper case.

-------------------------------------------------------
Date: Tue 12/21/2004 at 21:23       By: Michael Koch <mkoch>
I just commited a fix to CVS.Can you please try it and test if all things are 
fixed ? I will leave this bug open for now. You can close it when you verified 
all stuff works as expected.

-------------------------------------------------------
Date: Tue 12/21/2004 at 17:40       By: 0 <None>

Oh, another bug: CP setDisplayedMnemonic cares about
char case, Sun's does not.












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

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