RFR(S): 8168341: Gtests on JSon produce an enormous amount of hardly understandable output to stdout

Igor Ignatyev igor.ignatyev at oracle.com
Tue Nov 29 12:02:14 UTC 2016


Kirill,

still looks good to me

Thanks,
— Igor
> On Nov 29, 2016, at 2:58 PM, Kirill Zhaldybin <kirill.zhaldybin at oracle.com> wrote:
> 
> Dear all,
> 
> I found (unfortunately a bit late) that stringStream requires ResourceMark.
> 
> Here are a new WebRev: http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8168341/webrev.01/
> The only differences are added resource mark, #include "memory/resourceArea.hpp" and fixed indents.
> 
> Could you please let me know your opinion?
> 
> Thank you.
> 
> Regards, Kirill
> 
> On 25.11.2016 21:12, Kirill Zhaldybin wrote:
>> Dear all,
>> 
>> Could you please review this fix for 8168341?
>> 
>> I redirected output to stringStream instead of tty and appended it to an
>> assert message in case of failure.
>> 
>> WebRev: http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8168341/webrev.00/
>> CR: https://bugs.openjdk.java.net/browse/JDK-8168341
>> 
>> Thank you.
>> 
>> Regards, Kirill
> 



More information about the hotspot-compiler-dev mailing list