possible setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE) missing

Project:JNode Core
Component:Miscellaneous
Category:bug report
Priority:normal
Assigned:Unassigned
Status:patch (code needs review)
Description

The following Applications do not exit or finish when the window is closed. Tetris was fixed when I mentioned it to Fabien, however the following have the same problem.

AWTDemo, AWTFrameTest, AWTMenuTest, AWTTest, ConsoleTest, ImageTest, JInternalFrameTest, JTableTest, JTreeTest, MouseEventTest, RobotTest, TextFieldTest

The patch in 'unified diff'

Submitted by lsantha on Mon, 10/22/2007 - 20:03.

The patch in 'unified diff' format has been committed. Thanks.

#1

Submitted by GriffenJBS on Sat, 10/20/2007 - 23:31.
Status:active» patch (code needs review)

Fixes for AWTDemo.java

#2

Submitted by GriffenJBS on Sat, 10/20/2007 - 23:45.

Fixes for AWTFrameTest.

AWTMenuTest has already been fixed.

#3

Submitted by GriffenJBS on Sun, 10/21/2007 - 00:08.

Cleaning up AWTMenuTest it's problems were in the superclass AWTTest.

#4

Submitted by GriffenJBS on Sun, 10/21/2007 - 00:08.

Fixes for AWTTest

#5

Submitted by GriffenJBS on Sun, 10/21/2007 - 00:26.

Fixes for ConsoleTest

#6

Submitted by GriffenJBS on Sun, 10/21/2007 - 07:00.

I believe I have fixes for the rest of the ones I listed. Some needed setDefaultCloseOperation() as they were JFrame, others needed other solutions depending on each case. I also have updates for all the files in org.jnode.test.gui which should bring them all up to v1.5 compatibility.

Is that package named correctly, or should it be org.jnode.gui.test?

#7

Submitted by GriffenJBS on Sun, 10/21/2007 - 09:03.

Here it is all 32 diffs in one nice and neat file. Sorry about the multiple replies above. I promise never to do that again. Smiling I'll find other mistakes to make. This diff includes issues_75 thru issues_79 and 28 more which were not previously posted.

#8

Submitted by GriffenJBS on Sun, 10/21/2007 - 19:13.

Same diff as last time, hopefull with the correct format, the last one was a "context diff", this project seems to use "unified diff". Sorry for the confusion,

#9

Submitted by GriffenJBS on Mon, 10/22/2007 - 04:38.

If anyone needs help with Netbeans and JNode, I'm becomming an expert.