This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: [gui] [merge] java-gui-merge for may 26
- From: Michael Koch <konqueror at gmx dot de>
- To: java-patches at gcc dot gnu dot org
- Cc: Andreas Schwab <schwab at suse dot de>, graydon hoare <graydon at pobox dot com>, classpath at gnu dot org, gcc at gcc dot gnu dot org
- Date: Thu, 27 May 2004 14:28:36 +0200
- Subject: Re: [gui] [merge] java-gui-merge for may 26
- References: <40B58C84.5010207@pobox.com> <jer7t6ci6o.fsf@sykes.suse.de>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Am Donnerstag, 27. Mai 2004 11:50 schrieb Andreas Schwab:
> graydon hoare <graydon@pobox.com> writes:
> > hi,
> >
> > I've just completed the "monthly merge" from the java-gui-branch
> > to the gcc libjava trunk. the following patch contains the
> > changes:
> >
> >
> > http://people.redhat.com/graydon/java-gui-merge-may-26-2004.patch
>
> libjava does not build any more.
>
> /tmp/cvs/gcc-20040527/Build/gcc/gcj
> -B/tmp/cvs/gcc-20040527/Build/ia64-suse-linux/libjava/
> -B/tmp/cvs/gcc-20040527/Build/gcc/ --encoding=UTF-8 -Wno-deprecated
> -C -g -classpath '' -bootclasspath
> /tmp/cvs/gcc-20040527/Build/ia64-suse-linux/libjava:../../../libjav
>a -d /tmp/cvs/gcc-20040527/Build/ia64-suse-linux/libjava
> ../../../libjava/javax/swing/JWindow.java
> ../../../libjava/javax/swing/JWindow.java: In class
> `javax.swing.JWindow': ../../../libjava/javax/swing/JWindow.java:
> In constructor `()': ../../../libjava/javax/swing/JWindow.java:83:
> error: Can't access package-private constructor
> `java.awt.Window.<init>' from `javax.swing.JWindow'. super();
> ^
> 1 error
> make[2]: *** [javax/swing/JWindow.class] Error 1
I have just commited the attached patch with lets libjava bootstrap
again. Its not really tested and only to make libjava compile again.
Graydon: you should really consider to update gcj in java-gui-branch.
Michael
2004-05-27 Michael Koch <konqueror@gmx.de>
* javax/swing/JPopupMenu.java
(isVisible): Do not use visible directly.
(setVisible): Likewise.
* javax/swing/JWindow.java
(JWindow): call accessible constructor.
* javax/swing/RepaintManager.java
(paintDirtyRegions): Use public methods to obtain iterator.
- --
Homepage: http://www.worldforge.org/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)
iD8DBQFAtd70WSOgCCdjSDsRAhK3AKCRymFeUH11bsWdpPsafF/DifyHWQCfbwTj
WX4pzyDVowIzqxSsjPeUi5Y=
=A4q3
-----END PGP SIGNATURE-----
Index: javax/swing/JPopupMenu.java
===================================================================
RCS file: /cvs/gcc/gcc/libjava/javax/swing/JPopupMenu.java,v
retrieving revision 1.4
diff -u -b -B -r1.4 JPopupMenu.java
--- javax/swing/JPopupMenu.java 27 May 2004 06:17:39 -0000 1.4
+++ javax/swing/JPopupMenu.java 27 May 2004 12:25:18 -0000
@@ -471,7 +471,7 @@
*/
public boolean isVisible()
{
- return super.visible;
+ return super.isVisible();
}
/**
@@ -481,7 +481,7 @@
*/
public void setVisible(boolean visible)
{
- super.visible = visible;
+ super.setVisible(visible);
firePopupMenuWillBecomeVisible();
Index: javax/swing/JWindow.java
===================================================================
RCS file: /cvs/gcc/gcc/libjava/javax/swing/JWindow.java,v
retrieving revision 1.5
diff -u -b -B -r1.5 JWindow.java
--- javax/swing/JWindow.java 27 May 2004 06:17:39 -0000 1.5
+++ javax/swing/JWindow.java 27 May 2004 12:25:18 -0000
@@ -80,7 +80,7 @@
public JWindow()
{
- super();
+ super(null); // FIXME: This throws an exception.
}
// huuu ?
Index: javax/swing/RepaintManager.java
===================================================================
RCS file: /cvs/gcc/gcc/libjava/javax/swing/RepaintManager.java,v
retrieving revision 1.3
diff -u -b -B -r1.3 RepaintManager.java
--- javax/swing/RepaintManager.java 27 May 2004 06:17:39 -0000 1.3
+++ javax/swing/RepaintManager.java 27 May 2004 12:25:18 -0000
@@ -438,7 +438,7 @@
dirtyComponents.clear();
// step 2: paint those roots
- Iterator i = roots.iterator(AbstractMap.ENTRIES);
+ Iterator i = roots.values().iterator();
while(i.hasNext())
{
AbstractMap.BasicMapEntry ent = (AbstractMap.BasicMapEntry) i.next();