Using menu breaking my code. Help?

Question

I don't think this is a bug; I think I'm just missing something. Following is an abbreviated plug-n-play code snippet to reproduce the problem.
The problem is that I can make a JProgressBar work in a JFrame nicely if I do it on it's own, but if I try to execute it as a result of a menu-item click, it only displays the JFrame; the JProgressBar never shows up. Replace the progress bar with a button, label, whatever.
I'm thinking it has something to do with the menu needing to be dismissed or something, but I may be totally off base. Can't find any appropriate documentation. Just finished scanning through 810 bug reports. Need help. What am I missing?
Using j2sdk_1.4_03
Signed,
Feelin' dumb.
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class Test extends JFrame implements ActionListener
Test() {
super("Test");
JMenuBar jmb;
JMenu jm;
JMenuItem jmi;
setSize(300, 200);
jmb = new JMenuBar();
jm = new JMenu("File");
jmi = new JMenuItem("go");
jmi.setActionCommand("go");
jmi.addActionListener(this);
jm.add(jmi);
jmb.add(jm);
setJMenuBar(jmb);
addWindowListener(new WindowAdapter() {
public void windowClosing(WindowEvent e) {
dispose();
System.exit(0); }
setVisible(true);
public void actionPerformed(ActionEvent e) {
progressManager pm = new progressManager(0,10000000);
long loop = 0;
while(loop < 10000000) {
pm.setValue(loop++);
pm.close();
public static void main(String args[]) {
System.out.println("Starting");
// This is to show it works without menus... takes a couple
// seconds on my 1.6GHz PC, make the loop smaller if you don't
// want to wait so long.
progressManager pm = new progressManager(0,10000000);
long loop = 0;
while(loop < 10000000) {
pm.setValue(loop++);
pm.close();
Test t = new Test();
class progressManager {
JFrame jf;
JProgressBar jpb;
long min, max;
progressManager(long min, long max) {
this.min = min; this.max = max;
jf = new JFrame("Progress");
jpb = new JProgressBar(0, 100);
Container c = jf.getContentPane();
c.add(jpb);
jf.setBounds(100, 100, 200, 100);
jf.setVisible(true);
public void setValue(long value) {
jpb.setValue((int)(value * 100 / max));
public void close() {
jf.dispose();

Answer 1

Sorry to make this page so long. Here is the code properly formatted and updated as you suggest. The problem still exists, but should be more readable. Thank you very my for the help so far.
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class Test extends JFrame
  Test()
    super("Test");
    JMenuBar jmb;
    JMenu jm;
    JMenuItem jmi;
    setSize(300, 200);
    jmb = new JMenuBar();
    jm = new JMenu("File");
    jmi = new JMenuItem("go");
     //jmi.setActionCommand( "go" );
     jmi.addActionListener( new ActionListener() {
          public void actionPerformed( ActionEvent e ) {
               progressManager pm = new progressManager(0,10000000);
               long loop = 0;
               while(loop < 10000000) {
                    pm.setValue(loop++);
               pm.close();
    jm.add(jmi);
    jmb.add(jm);
    setJMenuBar(jmb);
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent e)
        dispose();
        System.exit(0);
    setVisible(true);
  public static void main(String args[])
    System.out.println("Starting");
    // This is to show it works without menus...
    progressManager pm = new progressManager(0,10000000);
    long loop = 0;
    while(loop < 10000000)
      pm.setValue(loop++);
    pm.close();
    Test t = new Test();
class progressManager
  JFrame jf;
  JProgressBar jpb;
  long min, max;
  progressManager(long min, long max)
    this.min = min; this.max = max;
    jf = new JFrame("Progress");
    jpb = new JProgressBar(0, 100);
    Container c = jf.getContentPane();
    c.add(jpb);
    jf.setBounds(100, 100, 200, 100);
    jf.setVisible(true);
  public void setValue(long value)
    jpb.setValue((int)(value * 100 / max));
  public void close()
    jf.dispose();
}