On Sun, Jan 18, 2009 at 4:19 AM, Rahul Akolkar <rahul.akol...@gmail.com> wrote:

>> +        } finally {
>> +            if (!successful) {
>> +                for (Iterator iterator = items.iterator(); 
>> iterator.hasNext();) {
>> +                    FileItem fileItem = (FileItem) iterator.next();
>> +                    try {
>> +                        fileItem.delete();
>> +                    } catch (Throwable e) {
>> +                        // ignore it
>> +                    }
>
>
> Catch the bits that makes sense to ignore here?

Don't know, whether I understand your question right, Rahul. But if
the question is, whether ignoring the exception in the finally clause
is fine: Yes, it is! If the flag "successful" isn't set, then another
exception was thrown previously, and we want to see that exception,
not a follow-up.

Jochen

-- 
I have always wished for my computer to be as easy to use as my
telephone; my wish has come true because I can no longer figure out
how to use my telephone.

    -- (Bjarne Stroustrup,
http://www.research.att.com/~bs/bs_faq.html#really-say-that
       My guess: Nokia E50)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to