sandygao    2002/08/29 12:41:29

  Modified:    java/src/org/apache/xerces/parsers XMLGrammarPreparser.java
  Log:
  It's better to use Hashtable#elements() to get all the values out of a hash
  table, than to call Hashtable#keys() to get all the keys, then use the get()
  method to get the corresponding values.
  
  Revision  Changes    Path
  1.5       +7 -9      
xml-xerces/java/src/org/apache/xerces/parsers/XMLGrammarPreparser.java
  
  Index: XMLGrammarPreparser.java
  ===================================================================
  RCS file: 
/home/cvs/xml-xerces/java/src/org/apache/xerces/parsers/XMLGrammarPreparser.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- XMLGrammarPreparser.java  26 Aug 2002 23:58:02 -0000      1.4
  +++ XMLGrammarPreparser.java  29 Aug 2002 19:41:29 -0000      1.5
  @@ -315,10 +315,9 @@
       // by a grammar loader of a particular type, it will have
       // to retrieve that loader and use the loader's setFeature method.
       public void setFeature(String featureId, boolean value) {
  -        Enumeration keys = fLoaders.keys();
  -        while(keys.hasMoreElements()){
  -            Object key = keys.nextElement();
  -            XMLGrammarLoader gl = (XMLGrammarLoader)fLoaders.get(key);
  +        Enumeration loaders = fLoaders.elements();
  +        while(loaders.hasMoreElements()){
  +            XMLGrammarLoader gl = (XMLGrammarLoader)loaders.nextElement();
               try {
                   gl.setFeature(featureId, value);
               } catch(Exception e) {
  @@ -335,10 +334,9 @@
       // <p> <strong>An application should use the explicit method
       // in this class to set "standard" properties like error handler etc.</strong>
       public void setProperty(String propId, Object value) {
  -        Enumeration keys = fLoaders.keys();
  -        while(keys.hasMoreElements()){
  -            Object key = keys.nextElement();
  -            XMLGrammarLoader gl = (XMLGrammarLoader)fLoaders.get(key);
  +        Enumeration loaders = fLoaders.elements();
  +        while(loaders.hasMoreElements()){
  +            XMLGrammarLoader gl = (XMLGrammarLoader)loaders.nextElement();
               try {
                   gl.setProperty(propId, value);
               } catch(Exception e) {
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to