Vadim Gritsenko wrote:
[EMAIL PROTECTED] wrote:

throw new ProcessingException("SAXException JspGenerator.generate()",e.getException());
} catch (IOException e) {
throw new ProcessingException("IOException JspGenerator.generate()",e);
- } catch (ProcessingException e) {
- throw e;



This code avoids unnecessary wrapping of ProcessingException into one more ProcessingException. What was the reason to remove it? It took some time to add all those small pieces to avoid excessive exception wrapping...


Vadim

Sorry, have been too fast here. I only saw the re-throw, which could be removed in general, but not if Exception is caught afterwards.


Joerg

Reply via email to