Re: EmailWorker not threadsafe?

2008-07-19 Thread Tim Ruppert
+1 On Jul 18, 2008, at 8:13 PM, Ean Schuessler wrote: I can fix the thread problem easily enough, just need to pass the attachment count as a parameter in the method call rather than having a static field. My problem was that it didn't seem to save attachments correctly. I have a rather

Re: EmailWorker not threadsafe?

2008-07-18 Thread Ean Schuessler
I can fix the thread problem easily enough, just need to pass the attachment count as a parameter in the method call rather than having a static field. My problem was that it didn't seem to save attachments correctly. I have a rather special case need where I handle nested trees of forwarded

Re: EmailWorker not threadsafe?

2008-07-17 Thread Ean Schuessler
I'll take that as a no. :-D - David E Jones [EMAIL PROTECTED] wrote: Sounds like a pernicious evil and should be corrected On Jul 16, 2008, at 5:54 PM, Ean Schuessler wrote: Is anyone making use of this code in a production setting? -- Ean Schuessler, CTO Brainfood.com [EMAIL

Re: EmailWorker not threadsafe?

2008-07-17 Thread David Jones
Actually, my guess is that people are using it in production, and either haven't been hit by the problem yet, or haven't noticed, or having noticed weren't sure what to do about it... Do you have a patch for this, or do we need a volunteer to knock this out? -David On Thu, 17 Jul 2008

Re: EmailWorker not threadsafe?

2008-07-17 Thread BJ Freeman
I reported this back last year. glad to know what was causing it. David Jones sent the following on 7/17/2008 12:58 PM: Actually, my guess is that people are using it in production, and either haven't been hit by the problem yet, or haven't noticed, or having noticed weren't sure what to do

EmailWorker not threadsafe?

2008-07-16 Thread Ean Schuessler
Is anyone using EmailWorker? I notice that it uses a class-wide static private int to keep track of how many attachments the email currently being stored has, which certainly isn't thread-safe and would almost certainly be tickled. It creates CommunicationEvents for me successfully but doesn't

Re: EmailWorker not threadsafe?

2008-07-16 Thread David E Jones
Sounds like a pernicious evil and should be corrected -David On Jul 16, 2008, at 5:54 PM, Ean Schuessler wrote: Is anyone using EmailWorker? I notice that it uses a class-wide static private int to keep track of how many attachments the email currently being stored has, which