Integrated: 8218857: Confusing overloads for os::open
Harold Seigel
hseigel at openjdk.java.net
Mon Jan 10 14:01:31 UTC 2022
On Fri, 7 Jan 2022 13:29:38 GMT, Harold Seigel <hseigel at openjdk.org> wrote:
> Please review this small change to resolve overload confusion by renaming "FILE* os::open(int fd, const char* mode)" to "FILE* os::fdopen(int fd, const char* mode)" in os.hpp. The change was tested by running Mach5 tiers 1-2 on Linux, Mac OS, and Windows, and Mach5 tiers 3-5 on Linux x64.
>
> Thanks, Harold
This pull request has now been integrated.
Changeset: 11d88ce8
Author: Harold Seigel <hseigel at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/11d88ce82efd72d3d63f7c7271c285cd21b01217
Stats: 10 lines in 6 files changed: 0 ins; 0 del; 10 mod
8218857: Confusing overloads for os::open
Reviewed-by: kbarrett, rehn
-------------
PR: https://git.openjdk.java.net/jdk/pull/6988
More information about the hotspot-dev
mailing list