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 ) ) );
     }
 
     /**


Reply via email to