RFR: 8048020 - Regression on java.util.logging.FileHandler

[Alan Bateman]

On 04/07/2014 19:10, Daniel Fuchs wrote:
> hmmm. yes - you're right. I should catch that to and break the loop in 
> that case.
> So that would become:
>
>   465                             } catch (NoSuchFileException x) {
>   466                                 // Race condition - retry once, and if that
>   467                                 // fails again just try the next name in
>   468                                 // the sequence.
>   469                                 continue;
>   470                             } catch (IOException x) {
>                                       // the file may not be writable for us.
>                                       // try the next name in the sequence
>                                       break;
>                                   }
>
> Thanks for the feedback Alan!
> I had missed those cases.
If that is the final change then I think I'm okay (no need to generate a 
new webrev). I do think it could be improved a bit more by dropping the 
isXXX checks but it's not important.

-Alan