RFR: 8342807: Update links in java.base to use https:// [v3]
Magnus Ihse Bursie
ihse at openjdk.org
Mon Jan 20 21:32:38 UTC 2025
On Mon, 20 Jan 2025 19:34:53 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> In ProcessImpl_md.c, we have adopted the Posix APIs and semantics for process handling.
>> I suggest removing the "unofficial" link, it does not include useful information at this point and is fragile.
>>
>> The current behavior always setting SIGCHLD to SIG_DFL is/has been reliable and does not depend on the behavior of the parent process. (Good)
>> @tstuefe May also be interested in reviewing this change.
>>
>> The previous sentence mentions Solaris and that should be removed at some point but out of scope for this PR.
>
>> In ProcessImpl_md.c, we have adopted the Posix APIs and semantics for process handling. I suggest removing the "unofficial" link, it does not include useful information at this point and is fragile.
>>
>> The current behavior always setting SIGCHLD to SIG_DFL is/has been reliable and does not depend on the behavior of the parent process. (Good) @tstuefe May also be interested in reviewing this change.
>
> https://pubs.opengroup.org/onlinepubs/9799919799/ seems to indicate that the "default action is to ignore the signal", which is unclear to me. I think it means that the signal will not cause invocation of a user handler or abort the process, but does this also mean that wait and waited won't work?
>
> In any case, I agree with @RogerRiggs that keeping this behavior is fine.
>
>>
>> The previous sentence mentions Solaris and that should be removed at some point but out of scope for this PR.
@tstuefe And you also agree that the link should be removed? (Which is what this PR is about)
-------------
PR Comment: https://git.openjdk.org/jdk/pull/21633#issuecomment-2603253756
More information about the net-dev
mailing list