RFR: 8324683: Unify AttachListener code for Posix platforms [v3]
Thomas Stuefe
stuefe at openjdk.org
Wed Apr 17 12:56:51 UTC 2024
On Tue, 16 Apr 2024 18:48:16 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:
>>> > Hi @SoniaZaldana, something is off. Did you forget to push the deletion of the linux variant?
>>>
>>> Hi @tstuefe, I deleted it and created a new file (attachListener_posix.cpp) but git picked it up as a file rename. Is this what you mean?
>>
>> Oh, okay. How annoying of git :)
>>
>> Okay then.
>
> @tstuefe: I guess you meant "Fine by me! I do think that the credentials checks could be moved out-of-line into something like pd_check_credentials to remove clutter from the main code." by "Johan's nits"?
> @TheRealMDoerr , @tstuefe, @SoniaZaldana
>
> Yeah, I really don't want the `Nix` naming. I only meant the `pd` functions, which Sonia can bring in if she wants to. Sorry Sonia!!!
Mhhm. Lets just finish this one. @jdksjolen feel free to do any subsequent renaming on the next cleanup patch you were planning.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18283#issuecomment-2061191283
More information about the hotspot-runtime-dev
mailing list