RFR: 8326433: Make file-local functions static in src/jdk.jdwp.agent/unix/native/libjdwp/exec_md.c [v2]

[Jiangli Zhou]

On Mon, 26 Feb 2024 22:15:00 GMT, Jiangli Zhou <jiangli at openjdk.org> wrote:

>> src/java.base/unix/native/libjava/childproc.h line 134:
>> 
>>> 132: int closeSafely(int fd);
>>> 133: int isAsciiDigit(char c);
>>> 134: int closeDescriptors(void);
>> 
>> It seems that most of the APIs in this file should be static. I don't think you should selectively deal with just one of them because of the conflict. Since this ends up being a more involved change, and is in a different component than the jdwp change, it should probably have a separate PR.
>
> @plummercj thanks for looking into this! Sounds good to make the additional local functions static in these files. Perhaps we can use two different FRs. I'll make the current one to handle the ones in src/jdk.jdwp.agent/unix/native/libjdwp/exec_md.c.

Filed https://bugs.openjdk.org/browse/JDK-8326714.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18013#discussion_r1503414392