Changeset 26168 for main/trunk


Ignore:
Timestamp:
2012-09-11T15:29:35+12:00 (12 years ago)
Author:
sjm84
Message:

Make sure that opened files are closed as soon as we are finished with them

File:
1 edited

Legend:

Unmodified
Added
Removed

  • main/trunk/greenstone3/src/java/org/greenstone/gsdl3/util/XMLConverter.java

    r25655 r26168  
    6767
    6868    protected EntityResolver resolver = null;
    69    
     69
    7070    /** xerces parser */
    7171    protected DOMParser parser = null;
    … …  
    147147            parser.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd", false);
    148148            parser.setFeature("http://apache.org/xml/features/dom/defer-node-expansion", false);
    149             if(resolver != null)
     149            if (resolver != null)
    150150            {
    151151                parser.setEntityResolver(this.resolver);
    … …  
    153153            parser.setErrorHandler(new ParseErrorHandler());
    154154            parser.parse(xml_source);
    155            
     155
    156156            Document doc = parser.getDocument();
    157157
     158            reader.close();
    158159            return doc;
    159160
    … …  
    178179            parser.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd", false);
    179180            parser.setFeature("http://apache.org/xml/features/dom/defer-node-expansion", false);
    180             if(resolver != null)
     181            if (resolver != null)
    181182            {
    182183                parser.setEntityResolver(this.resolver);
    … …  
    184185            parser.setErrorHandler(new ParseErrorHandler());
    185186            parser.parse(xml_source);
    186            
     187
    187188            Document doc = parser.getDocument();
    188189
     190            reader.close();
    189191            return doc;
    190192
    … …  
    196198        return null;
    197199    }
    198    
     200
    199201    /** returns a DOM Document */
    200202    public Document getDOM(File in)
    … …  
    204206            FileReader reader = new FileReader(in);
    205207            InputSource xml_source = new InputSource(reader);
    206            
     208
    207209            DOMParser parser = new DOMParser();
    208210            parser.setFeature("http://xml.org/sax/features/validation", false);
    209211            parser.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd", false);
    210212            parser.setFeature("http://apache.org/xml/features/dom/defer-node-expansion", false);
    211             if(resolver != null)
     213            if (resolver != null)
    212214            {
    213215                parser.setEntityResolver(this.resolver);
    … …  
    215217            parser.setErrorHandler(new ParseErrorHandler());
    216218            parser.parse(xml_source);
    217            
     219
    218220            Document doc = parser.getDocument();
     221
     222            reader.close();
    219223            return doc;
    220224
    … …  
    241245            parser.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd", false);
    242246            parser.setFeature("http://apache.org/xml/features/dom/defer-node-expansion", false);
    243             if(resolver != null)
     247            if (resolver != null)
    244248            {
    245249                parser.setEntityResolver(this.resolver);
    … …  
    247251            parser.setErrorHandler(new ParseErrorHandler());
    248252            parser.parse(xml_source);
    249            
     253
    250254            Document doc = parser.getDocument();
    251255
     256            isr.close();
    252257            return doc;
    253258
Note: See TracChangeset for help on using the changeset viewer.