[Common Configuration] Bad Exception Handling in ConfigurationFactory

2010-09-14 Thread Nicolás Meléndez
Hi there, i was looking code at ConfigurationFactory and maybe a find a problem, what do you think? in class ConfigurationFactory, line 184: if digester.parse(input); throws an exception, the input stream doesn't close, and a process will leak of resources. i think there should be a finally

Re: [Common Configuration] Bad Exception Handling in ConfigurationFactory

2010-09-14 Thread Oliver Heger
Am 14.09.2010 16:48, schrieb Nicolás Meléndez: Hi there, i was looking code at ConfigurationFactory and maybe a find a problem, what do you think? in class ConfigurationFactory, line 184: if digester.parse(input); throws an exception, the input stream doesn't close, and a process will leak of