You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xerces.apache.org by mr...@apache.org on 2008/12/07 07:07:56 UTC

svn commit: r724088 - in /xerces/java/trunk/src/org/apache/xerces: impl/XMLEntityManager.java xinclude/XIncludeTextReader.java

Author: mrglavas
Date: Sat Dec  6 22:07:55 2008
New Revision: 724088

URL: http://svn.apache.org/viewvc?rev=724088&view=rev
Log:
Now that we've moved to JDK 1.3 call setInstanceFollowRedirects() directly instead of through reflection.

Modified:
    xerces/java/trunk/src/org/apache/xerces/impl/XMLEntityManager.java
    xerces/java/trunk/src/org/apache/xerces/xinclude/XIncludeTextReader.java

Modified: xerces/java/trunk/src/org/apache/xerces/impl/XMLEntityManager.java
URL: http://svn.apache.org/viewvc/xerces/java/trunk/src/org/apache/xerces/impl/XMLEntityManager.java?rev=724088&r1=724087&r2=724088&view=diff
==============================================================================
--- xerces/java/trunk/src/org/apache/xerces/impl/XMLEntityManager.java (original)
+++ xerces/java/trunk/src/org/apache/xerces/impl/XMLEntityManager.java Sat Dec  6 22:07:55 2008
@@ -24,7 +24,6 @@
 import java.io.OutputStream;
 import java.io.Reader;
 import java.io.StringReader;
-import java.lang.reflect.Method;
 import java.net.HttpURLConnection;
 import java.net.URL;
 import java.net.URLConnection;
@@ -984,7 +983,7 @@
                         // set preference for redirection
                         followRedirects = httpInputSource.getFollowHTTPRedirects();
                         if (!followRedirects) {
-                            setInstanceFollowRedirects(urlConnection, followRedirects);
+                            urlConnection.setInstanceFollowRedirects(followRedirects);
                         }
                     }
                     
@@ -1861,19 +1860,6 @@
         
     } // expandSystemIdStrictOff(String,String):String
     
-    /**
-     * Attempt to set whether redirects will be followed for an <code>HttpURLConnection</code>.
-     * This may fail on earlier JDKs which do not support setting this preference.
-     */
-    public static void setInstanceFollowRedirects(HttpURLConnection urlCon, boolean followRedirects) {
-        try {
-            Method method = HttpURLConnection.class.getMethod("setInstanceFollowRedirects", new Class[] {Boolean.TYPE});
-            method.invoke(urlCon, new Object[] {followRedirects ? Boolean.TRUE : Boolean.FALSE});
-        }
-        // setInstanceFollowRedirects doesn't exist.
-        catch (Exception exc) {}
-    }
-    
     public static OutputStream createOutputStream(String uri) throws IOException {
         // URI was specified. Handle relative URIs.
         String expanded = XMLEntityManager.expandSystemId(uri, null, true);

Modified: xerces/java/trunk/src/org/apache/xerces/xinclude/XIncludeTextReader.java
URL: http://svn.apache.org/viewvc/xerces/java/trunk/src/org/apache/xerces/xinclude/XIncludeTextReader.java?rev=724088&r1=724087&r2=724088&view=diff
==============================================================================
--- xerces/java/trunk/src/org/apache/xerces/xinclude/XIncludeTextReader.java (original)
+++ xerces/java/trunk/src/org/apache/xerces/xinclude/XIncludeTextReader.java Sat Dec  6 22:07:55 2008
@@ -141,7 +141,7 @@
                     // set preference for redirection
                     boolean followRedirects = httpInputSource.getFollowHTTPRedirects();
                     if (!followRedirects) {
-                        XMLEntityManager.setInstanceFollowRedirects(urlConnection, followRedirects);
+                        urlConnection.setInstanceFollowRedirects(followRedirects);
                     }
                 }
                 



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@xerces.apache.org
For additional commands, e-mail: commits-help@xerces.apache.org