[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: AWT regression?
From: |
Grzegorz B. Prokopski |
Subject: |
Re: AWT regression? |
Date: |
Thu, 29 Apr 2004 09:42:47 -0400 |
On Wed, 2004-04-28 at 12:46, Thomas Fitzsimmons wrote:
> On Sat, 2004-04-24 at 18:52, Grzegorz B. Prokopski wrote:
> > Hi all,
> >
> > We've just imported fresh GNU CP as of Fri Apr 23 and running SymbolTest
> > program, that previously worked nicely, as documented ex. at
> > http://www.sablevm.org/screenshots , now gives this:
> >
> > $ ~/work/bin/java-sablevm SymbolTest
> > (:19347): Gtk-CRITICAL **: file gtkwidget.c: line 4248
> > (gtk_widget_modify_fg): assertion `GTK_IS_WIDGET (widget)' failed
> > (:19347): Gtk-CRITICAL **: file gtkwidget.c: line 4248
> > (gtk_widget_modify_fg): assertion `GTK_IS_WIDGET (widget)' failed
> > (:19347): Gtk-CRITICAL **: file gtkwidget.c: line 4248
> > (gtk_widget_modify_fg): assertion `GTK_IS_WIDGET (widget)' failed
> > java.lang.NullPointerException
> > at java.applet.Applet.getDimensions (Applet.java:469)
> > at java.applet.Applet.preferredSize (Applet.java:486)
> > at java.awt.Container.getPreferredSize (Container.java:576)
> > at java.awt.BorderLayout.calcCompSize (BorderLayout.java:647)
> > at java.awt.BorderLayout.calcSize (BorderLayout.java:692)
> > at java.awt.BorderLayout.preferredLayoutSize (BorderLayout.java:454)
> > at java.awt.Container.preferredSize (Container.java:589)
> > at java.awt.Container.getPreferredSize (Container.java:576)
> > at java.awt.Window.pack (Window.java:206)
> > at SymbolTest.main (SymbolTest.java:100)
> > at java.lang.VirtualMachine.invokeMain (VirtualMachine.java)
> > at java.lang.VirtualMachine.main (VirtualMachine.java:88)
> >
>
> I recently changed size reporting in Applet.java. I made applets report
> their preferred size as the width and height attributes given in the
> applet tag. However, the current code assumes that an applet stub is
> present that can be queried for these attributes. The NPE you're seeing
> is likely caused by the stub not having been set.
>
> The attached patch should fix the problem, though I haven't tested it.
> Can you try it out?
Hi Thomas,
I got a report (had no time to do it myself :-( ) that your patch
actually fixed the problem. I guess it should go into GNU CP CVS ASAP
and 0.09 release too.
Thanks for the help,
Grzegorz B. Prokopski