Wow.. you are right.... We should catch this with the
IllegalArgumentException ..
I not like to use catch (Exception) ..
So i think the patch is not complete like it is..
bye
Norman
Stefano Bagnara schrieb:
Norman Maurer wrote:
I still think you missunderstood it ;-) I only change the "throws
Exception" to "throws ContextException"..
Thats all ..
You also changed this:
---
james/server/trunk/src/java/org/apache/james/context/AvalonFileSystem.java
(original)
+++
james/server/trunk/src/java/org/apache/james/context/AvalonFileSystem.java
Sat Oct 7 10:58:04 2006
@@ -47,8 +47,6 @@
return AvalonContextUtilities.getFile(context, fileURL);
} catch (ContextException e) {
throw new FileNotFoundException("Context exception:
"+e.getMessage());
- } catch (Exception e) {
- throw new FileNotFoundException(e.getMessage());
}
}
And the catch(Exception e) is the block that cought the
IllegalArgumentException that now is no more cought: did I understand?
Stefano
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
!EXCUBATOR:1,45281b5a53071024755060!
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
