RFR: 8330989: unify os::create_binary_file across platforms [v2]

Matthias Baesken mbaesken at openjdk.org
Mon Apr 29 11:28:04 UTC 2024


On Wed, 24 Apr 2024 08:03:52 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> Seems os::create_binary_file can be unified across Posix platforms. Windows is also rather similar and can be included.
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   unify windows as well

okay the let's NOT use `os::open` .
Regarding recommended  `_open` vs,  `::open`  on Windows, we used `::open`  before in `create_binary_file`   , so this should be fine and is just the same code moved to os.cpp . 
Regarding the flags, they exist on Windows with and without leading ` _`  - any reason to go to the versions with leading ` _`  ?
I can do so if you want but it would only add ifdefs to the code .

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

PR Comment: https://git.openjdk.org/jdk/pull/18918#issuecomment-2082469115


More information about the hotspot-runtime-dev mailing list