On Wed, 2006-02-08 at 08:47 +0000, Chris Burdess wrote:
> Lillian Angel wrote:
> > I added this check to prevent several assertion errors in GdkGraphics.
> >
> > 2006-02-07 Lillian Angel <[EMAIL PROTECTED]>
> >
> > * gnu/java/awt/peer/gtk/GtkComponentPeer.java
> > (handleEvent): Added check. Should not paint or update the
> > component if it's width and height are both 0.
>
> Surely that should be "if either its height or its width are less
> than 1"?
Yes, Fixed.
2006-02-08 Lillian Angel <[EMAIL PROTECTED]>
* gnu/java/awt/peer/gtk/GtkComponentPeer.java
(handleEvent): Fixed check to determine if height or
width is less than 1.
Index: gnu/java/awt/peer/gtk/GtkComponentPeer.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/java/awt/peer/gtk/GtkComponentPeer.java,v
retrieving revision 1.99
diff -u -r1.99 GtkComponentPeer.java
--- gnu/java/awt/peer/gtk/GtkComponentPeer.java 7 Feb 2006 19:11:38 -0000 1.99
+++ gnu/java/awt/peer/gtk/GtkComponentPeer.java 8 Feb 2006 14:33:44 -0000
@@ -308,18 +308,20 @@
{
Graphics g = getGraphics();
+ if (awtComponent.getWidth() < 1 || awtComponent.getHeight() < 1)
+ break;
+
// Some peers like GtkFileDialogPeer are repainted by Gtk itself
if (g == null)
break;
g.setClip(((PaintEvent) event).getUpdateRect());
- if (awtComponent.getWidth() > 0 && awtComponent.getHeight() > 0)
- {
- if (id == PaintEvent.PAINT)
- awtComponent.paint(g);
- else
- awtComponent.update(g);
- }
+
+ if (id == PaintEvent.PAINT)
+ awtComponent.paint(g);
+ else
+ awtComponent.update(g);
+
g.dispose();
}
catch (InternalError e)
