RFR: 8176717: GC log file handle leaked to child processes
thomas.stuefe at gmail.com
Fri Mar 30 11:16:47 UTC 2018
looks okay. I stated my reservations against adding this function and the
platform specific code in its current form before, but I can certainly live
with this fix. It follows what we did in os::open(), so it is consistent in
Thanks for fixing this!
(btw, I am not sure which of my "2" Per meant, since I accidentally
miscounted my points in the earlier mail.)
On Thu, Mar 29, 2018 at 5:41 PM, Leo Korinth <leo.korinth at oracle.com> wrote:
> On 23/03/18 20:03, Leo Korinth wrote:
>> Cross-posting this to both hotspot-dev and hotspot-runtime-dev to get
>> more attention. Sorry.
>> Original mail conversation can be found here:
>> I need feedback to know how to continue.
> Below is a new webrev with Thomas' and Per's suggested name change
> from: os::fopen_retain(const char* path, const char* mode)
> to: os::fopen(const char* path, const char* mode)
> Full webrev:
> Review and/or comments please!
More information about the hotspot-dev