Re: RFR(XS): 8168412 Reduce buffering in jtreg timeouthandler

2016-10-20 Thread Marcus Larsson

Hi,


On 10/20/2016 02:03 PM, Staffan Larsen wrote:

Hi,

Please review this small patch to the jtreg timeouthandler to remove some 
buffering. We create PrintWriters in a couple of places and this patch adds the 
second parameter to the constructor to enable automatic flushing of the 
PrintWriter.

webrev: http://cr.openjdk.java.net/~sla/8168412/webrev.00 



Looks good!

Thanks,
Marcus


bug: https://bugs.openjdk.java.net/browse/JDK-8168412 


Thanks,
/Staffan





RFR(XS): 8168412 Reduce buffering in jtreg timeouthandler

2016-10-20 Thread Staffan Larsen
Hi,

Please review this small patch to the jtreg timeouthandler to remove some 
buffering. We create PrintWriters in a couple of places and this patch adds the 
second parameter to the constructor to enable automatic flushing of the 
PrintWriter.

webrev: http://cr.openjdk.java.net/~sla/8168412/webrev.00 

bug: https://bugs.openjdk.java.net/browse/JDK-8168412 


Thanks,
/Staffan