Re: RFR: 6633375: FileOutputStream_md.c should be merged into FileOutputStream.c

2021-06-24 Thread Brian Burkhalter
On Thu, 24 Jun 2021 21:20:29 GMT, Naoto Sato wrote: > Fine by me. Thanks! - PR: https://git.openjdk.java.net/jdk/pull/4589

Re: RFR: 6633375: FileOutputStream_md.c should be merged into FileOutputStream.c

2021-06-24 Thread Naoto Sato
On Thu, 24 Jun 2021 20:01:27 GMT, Brian Burkhalter wrote: > Merge identical Unix and Windows versions of FileOutputStream_md.c into > single, common FileOutputStream.c. Fine by me. - Marked as reviewed by naoto (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/4589

Re: RFR: 6633375: FileOutputStream_md.c should be merged into FileOutputStream.c

2021-06-24 Thread Brian Burkhalter
On Thu, 24 Jun 2021 20:01:27 GMT, Brian Burkhalter wrote: > Merge identical Unix and Windows versions of FileOutputStream_md.c into > single, common FileOutputStream.c. The `fileOpen()`, `writeSingle()`, and `writeBytes()` functions called here all expect to receive a field ID. This might be

Re: RFR: 6633375: FileOutputStream_md.c should be merged into FileOutputStream.c

2021-06-24 Thread Naoto Sato
On Thu, 24 Jun 2021 20:01:27 GMT, Brian Burkhalter wrote: > Merge identical Unix and Windows versions of FileOutputStream_md.c into > single, common FileOutputStream.c. Looks good. Would you consider removing `fos_fd` and passing it as a `jlong` as suggested in the issue? - PR:

RFR: 6633375: FileOutputStream_md.c should be merged into FileOutputStream.c

2021-06-24 Thread Brian Burkhalter
Merge identical Unix and Windows versions of FileOutputStream_md.c into single, common FileOutputStream.c. - Commit messages: - 6633375: FileOutputStream_md.c should be merged into FileOutputStream.c Changes: https://git.openjdk.java.net/jdk/pull/4589/files Webrev: