RFR: 8308252: Refactor line-by-line file reading code [v2]

David Holmes dholmes at openjdk.org
Wed May 17 04:04:43 UTC 2023


On Wed, 17 May 2023 04:02:20 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> I extracted the `get_line()` code from `CompileReplay` and put it in a utility class so that it can be used by `ClassListParser` as well. A few notable changes:
>> 
>> - Simplified the API
>> - Changed the buffer size to a size_t
>> - Added size overflow and OOM checks
>> - Brought over the `fdopen` logic from `ClassListParser` for handling long path names on Windows. (I don't know how valid this is nowadays, but I don't want to drop it in a refactoring PR).
>
> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
> 
>   more clean up

src/hotspot/share/utilities/lineReader.cpp line 64:

> 62: // Returns nullptr if we have reached EOF.
> 63: // \n is treated as the line separator
> 64: // All occurrences of \r are stripper.

s/stripper/stripped/

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14025#discussion_r1195903050


More information about the hotspot-dev mailing list