On Sun, Jan 18, 2009 at 8:24 AM, Jochen Wiedmann
<jochen.wiedm...@gmail.com> wrote:
> 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.
<snip/>
Similar to SCXML-103 [1] -- the above may be flagged for the same reason.
-Rahul
[1] https://issues.apache.org/jira/browse/SCXML-103
> 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
