Fixing more warnings

Michael Koch <[email protected]>
Newsgroups gmane.comp.java.classpath.extensions.xml
Message-ID <[email protected]>
Hi list,


When compiling the GNU jaxp included in classpath with jikes gives some 
warnings:

Issued 4 semantic warnings compiling 
"/home/mkoch/src/classpath/external/jaxp/source/gnu/xml/pipeline/XIncludeFilter.java":

   460.             setContentHandler (this);
                    <---------------------->
*** Caution: Ambiguous reference to member named "setContentHandler" 
inherited from type "gnu/xml/pipeline/EventFilter" but also declared or 
inherited in the enclosing type "gnu/xml/pipeline/XIncludeFilter". Explicit 
qualification is required.


   461.             setProperty (LEXICAL_HANDLER, this);
                    <--------------------------------->
*** Caution: Ambiguous reference to member named "setProperty" inherited from 
type "gnu/xml/pipeline/EventFilter" but also declared or inherited in the 
enclosing type "gnu/xml/pipeline/XIncludeFilter". Explicit qualification is 
required.


   464.             setDTDHandler (null);
                    <------------------>
*** Caution: Ambiguous reference to member named "setDTDHandler" inherited 
from type "gnu/xml/pipeline/EventFilter" but also declared or inherited in 
the enclosing type "gnu/xml/pipeline/XIncludeFilter". Explicit qualification 
is required.


   465.             setProperty (DECL_HANDLER, null);
                    <------------------------------>
*** Caution: Ambiguous reference to member named "setProperty" inherited from 
type "gnu/xml/pipeline/EventFilter" but also declared or inherited in the 
enclosing type "gnu/xml/pipeline/XIncludeFilter". Explicit qualification is 
required.


The attached patch fixes it for classpath but I think the patch is really 
easy to port to GNU jaxp CVS. I looked into GNU jaxp CVS and the fix is 
missing there too.


Michael

_______________________________________________
Classpathx-xml mailing list
[email protected]
http://mail.gnu.org/mailman/listinfo/classpathx-xml
jaxp.diff (text/x-diff, 1.5 KB)
Index: ChangeLog
===================================================================
RCS file: /cvsroot/classpath/classpath/ChangeLog,v
retrieving revision 1.1572
diff -u -b -B -r1.1572 ChangeLog
--- ChangeLog	23 Oct 2003 10:32:06 -0000	1.1572
+++ ChangeLog	23 Oct 2003 16:04:31 -0000
@@ -1,3 +1,8 @@
+2003-10-23  Michael Koch  <[email protected]>
+
+	* external/jaxp/source/gnu/xml/pipeline/XIncludeFilter.java
+	(Scrubber.Scrubber): Call methods of super class.
+
 2003-10-23  Sascha Brawer  <[email protected]>
 
         * java/awt/geom/CubicCurve2D.java (contains): Docfix for URL of embedded drawing.
Index: external/jaxp/source/gnu/xml/pipeline/XIncludeFilter.java
===================================================================
RCS file: /cvsroot/classpath/classpath/external/jaxp/source/gnu/xml/pipeline/XIncludeFilter.java,v
retrieving revision 1.1.1.1
diff -u -b -B -r1.1.1.1 XIncludeFilter.java
--- external/jaxp/source/gnu/xml/pipeline/XIncludeFilter.java	1 Feb 2003 02:10:22 -0000	1.1.1.1
+++ external/jaxp/source/gnu/xml/pipeline/XIncludeFilter.java	23 Oct 2003 16:04:31 -0000
@@ -457,12 +457,12 @@
 	    super (f);
 
 	    // process all content events
-	    setContentHandler (this);
-	    setProperty (LEXICAL_HANDLER, this);
+	    super.setContentHandler (this);
+	    super.setProperty (LEXICAL_HANDLER, this);
 
 	    // drop all DTD events
-	    setDTDHandler (null);
-	    setProperty (DECL_HANDLER, null);
+	    super.setDTDHandler (null);
+	    super.setProperty (DECL_HANDLER, null);
 	}
 
 	// maintain proxy locator
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.