You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/06/23 22:15:53 UTC

svn commit: r1139059 - /commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/Digester.java

Author: simonetripodi
Date: Thu Jun 23 20:15:53 2011
New Revision: 1139059

URL: http://svn.apache.org/viewvc?rev=1139059&view=rev
Log:
removed duplicated code, parse(InputSource) is enough to cover all the cases

Modified:
    commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/Digester.java

Modified: commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/Digester.java
URL: http://svn.apache.org/viewvc/commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/Digester.java?rev=1139059&r1=1139058&r2=1139059&view=diff
==============================================================================
--- commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/Digester.java (original)
+++ commons/proper/digester/trunk/src/main/java/org/apache/commons/digester3/Digester.java Thu Jun 23 20:15:53 2011
@@ -1549,12 +1549,10 @@ public class Digester
             throw new IllegalArgumentException( "File to parse is null" );
         }
 
-        configure();
         InputSource input = new InputSource( new FileInputStream( file ) );
         input.setSystemId( file.toURI().toURL().toString() );
-        getXMLReader().parse( input );
-        cleanup();
-        return ( this.<T> getRoot() );
+
+        return ( this.<T> parse( input ) );
     }
 
     /**
@@ -1599,11 +1597,7 @@ public class Digester
             throw new IllegalArgumentException( "InputStream to parse is null" );
         }
 
-        configure();
-        InputSource is = new InputSource( input );
-        getXMLReader().parse( is );
-        cleanup();
-        return ( this.<T> getRoot() );
+        return ( this.<T> parse( new InputSource( input ) ) );
     }
 
     /**
@@ -1624,11 +1618,7 @@ public class Digester
             throw new IllegalArgumentException( "Reader to parse is null" );
         }
 
-        configure();
-        InputSource is = new InputSource( reader );
-        getXMLReader().parse( is );
-        cleanup();
-        return ( this.<T> getRoot() );
+        return ( this.<T> parse( new InputSource( reader ) ) );
     }
 
     /**
@@ -1649,11 +1639,7 @@ public class Digester
             throw new IllegalArgumentException( "String URI to parse is null" );
         }
 
-        configure();
-        InputSource is = createInputSourceFromURL( uri );
-        getXMLReader().parse( is );
-        cleanup();
-        return ( this.<T> getRoot() );
+        return ( this.<T> parse( createInputSourceFromURL( uri ) ) );
     }
 
     /**
@@ -1675,11 +1661,7 @@ public class Digester
             throw new IllegalArgumentException( "URL to parse is null" );
         }
 
-        configure();
-        InputSource is = createInputSourceFromURL( url );
-        getXMLReader().parse( is );
-        cleanup();
-        return ( this.<T> getRoot() );
+        return ( this.<T> parse( createInputSourceFromURL( url ) ) );
     }
 
     /**