RFR: 8345249: Apply some conservative cleanups in FileURLConnection [v2]

Eirik Bjørsnøs eirbjo at openjdk.org
Wed Jan 22 14:37:48 UTC 2025


On Sat, 30 Nov 2024 18:39:01 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

>> Please review this PR which applies various cleanups to `sun.net.www.protocol.file.FileURLConnection`.
>> 
>> This class is known to be an _old, intricate, and hard to maintain piece of code._  However, there are some relatively straightforward and safe cleanups possible which improve readability and makes it easier to reason about what's going on in this class.
>> 
>> In this PR, I have chosen to make each individual small change a separate commit. This to assist review of each individual change, which can otherwise disappear a bit when reviewing the PR as a whole. 
>> 
>> A detailed listing of each commit follows in a separate comment.
>> 
>> This is a pure cleanup PR, no tests are added or updated in this PR.
>
> Eirik Bjørsnøs has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Replace String::valueOf with Long::toString

Thanks for your review Daniel!

Since this PR had no activity since early December, I'll let it linger for a day or two before integration, just in case there are any lingering concerns.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/22459#issuecomment-2607409565


More information about the net-dev mailing list