RFR: 8324683: Unify AttachListener code for Posix platforms [v3]
Johan Sjölen
jsjolen at openjdk.org
Wed Apr 17 09:36:02 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!!!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18283#issuecomment-2060827589
More information about the hotspot-runtime-dev
mailing list