Soeren Hilmer wrote:
>
>
> Okay, it looks like FetchMail has a check for this that goes like:
>
> // if we are already fetching then just return
> if (fetching) return;
> fetching = true;
>
> ...
> fetching = false;
>
> May I suggest though the more robust scheme:
> try {
> // if we are already fetching then just return
> if (fetching) return;
> fetching = true;
>
> ...
> } finally {
> fetching = false;
> }
>
> This will ensure that fetching is set to false, even if some
> runtime exception
> is thrown, and thereby ensuring that FetchMail is actually
> fetching after
> that.
Soeren
You are looking at an earlier version of fetchmail. The latest is here...
http://cvs.apache.org/viewcvs.cgi/james-server/src/java/org/apache/james/fet
chmail/?only_with_tag=branch_2_1_fcs#dirlist
In this version, the targetTriggered() method already includes the scheme
you suggest.
-- Steve
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
