RE: [PATCH]mbox mail repository v3

2003-08-29 Thread Jason Webb
-Original Message- From: Noel J. Bergman [mailto:[EMAIL PROTECTED] Sent: 28 August 2003 18:44 To: James Developers List Subject: RE: [PATCH]mbox mail repository v3 Jason. Not attached. The mailing list is probably filtering. What was the extension you used

RE: [PATCH]mbox mail repository v3

2003-08-29 Thread Jason Webb
-Original Message- From: Noel J. Bergman [mailto:[EMAIL PROTECTED] Sent: 29 August 2003 14:38 To: James Developers List Subject: RE: [PATCH]mbox mail repository v3 I have already committed changes, so you might want to check what I've done, and see if there are any

RE: [PATCH]mbox mail repository v3

2003-08-28 Thread Jason Webb
As requested the patches using the unified format. The mbox repository now has the corrected path code as well. -- Jason - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]

RE: [PATCH]mbox mail repository v3

2003-08-27 Thread Noel J. Bergman
Jason, Please remember to use diff -u when submitting patches. --- Noel - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]

[PATCH]mbox mail repository v3

2003-08-26 Thread Jason Webb
As discussed I have changed the usage of RuntimeException to use MessagingException where appropriate. The POP3Handler now calls the remove(CollectionMailImpl) method rather than call remove(String). I've also tidied up some of the Javadoc in POP3Handler as well. The other users of

RE: [PATCH]mbox mail repository v3

2003-08-26 Thread Noel J. Bergman
Any comments? This is an issue I ran into in the v2 code last night. You have this: mboxFile = destination.substring(mbox://.length(), destination.lastIndexOf(/)); That doesn't work with mbox:///home/noel/test-mbox, which the code stripped to /home/noel. Avalon may append user/ when making

RE: [PATCH]mbox mail repository v3

2003-08-26 Thread Noel J. Bergman
Jason, This code: try { // Attempt to read the TO field and see if it errors toAddr = mimeMessage.getRecipients(javax.mail.Message.RecipientType.TO).toString(); } catch (Exception e) { // It has errored, so time for plan B // use the from field I suppose try {