RFR [XS]: 8219630: cleanup hotspot ostream.cpp

Baesken, Matthias matthias.baesken at sap.com
Tue Feb 26 09:05:14 UTC 2019


Hi Thomas, thanks for the review !

>
>- you could remove the whole destructor, since it does nothing of value anymore.
>- you could use initializer list instead in the ctor:
> fdStream(int fd = -1) : _fd(fd) {}

Sure,  new webrev :

http://cr.openjdk.java.net/~mbaesken/webrevs/8219630.1/


Best regards, Matthias


From: Thomas Stüfe <thomas.stuefe at gmail.com>
Sent: Dienstag, 26. Februar 2019 09:13
To: Baesken, Matthias <matthias.baesken at sap.com>
Cc: hotspot-dev at openjdk.java.net
Subject: Re: RFR [XS]: 8219630: cleanup hotspot ostream.cpp

Looks good.

Good as it is, but since you are touching the code:

- you could use initializer list instead in the ctor:
 fdStream(int fd = -1) : _fd(fd) {}

- you could remove the whole destructor, since it does nothing of value anymore.

Cheers Thomas



On Tue, Feb 26, 2019 at 9:02 AM Baesken, Matthias <matthias.baesken at sap.com<mailto:matthias.baesken at sap.com>> wrote:
Hello, please review  this small patch .

In the review process of 8218811 it  has been noticed that ostream.cpp  can be cleaned up a bit ;  I want to address  the cleanups with this change .


Bug/webrev :

https://bugs.openjdk.java.net/browse/JDK-8219630

http://cr.openjdk.java.net/~mbaesken/webrevs/8219630.0/


Thanks, Matthias


More information about the hotspot-dev mailing list