RFR(S): 6885993: Named Thread: introduce print() and print_on(outputStream* st) methods
Christian Thalinger
christian.thalinger at oracle.com
Wed Apr 30 22:58:59 UTC 2014
I’d prefer to have the method body of:
+ virtual void print_on(outputStream* st) const {
in the .cpp file.
Otherwise this is very nice and looks good.
On Apr 30, 2014, at 2:47 AM, Zhengyu Gu <zhengyu.gu at oracle.com> wrote:
> This is more of code cleanup. Instead of duplicating similar code all over subclasses of NamedThread, NamedThread should implements print_on(outputStream* str) and its subclasses should only set "name" to use default the implementation.
>
> Webrev: http://cr.openjdk.java.net/~zgu/6885993/webrev.00/
> Bug: https://bugs.openjdk.java.net/browse/JDK-6885993
>
> Tests:
> - JPRT
> - Ran a Java program on Linux x64, do "CTRL-\" to dump Java stacks, make sure the output remains unchange.
>
> Thanks,
>
> -Zhengyu
More information about the hotspot-dev
mailing list