Changeset 26168

Show
Ignore:
Timestamp:
11.09.2012 15:29:35 (7 years ago)
Author:
sjm84
Message:

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

Files:
1 modified

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