On Wed, 8 Feb 2023 14:15:09 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:
>> [JDK-8297211](https://bugs.openjdk.org/browse/JDK-8297211) (#11258) missed >> one case with date headers: `last-modified`, `date`, `expires`. We can avoid >> expensive NPE. > > Andrey Turbanov has updated the pull request incrementally with one > additional commit since the last revision: > > 8301964: Expensive fillInStackTrace operation in > HttpURLConnection.getLastModified when no last-modified in response > > update copyright year The changes looks okay, I just wonder why the try-catch around Date.parse catches Exception, should it be IllegalArgumentException? ------------- Marked as reviewed by alanb (Reviewer). PR: https://git.openjdk.org/jdk/pull/12451
