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