RFR: 8048020 - Regression on java.util.logging.FileHandler (original) (raw)

Alan Bateman Alan.Bateman at oracle.com
Fri Jul 4 18:24:40 UTC 2014

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

More information about the core-libs-dev mailing list