RFR(L): 8156852: Convert JSON_test to Gtest

[Pavel Punegov]

Kirill and Vladimir,

Fixed copyright year and long lines.
New webrev: http://cr.openjdk.java.net/~ppunegov/8156852/webrev.01/ <http://cr.openjdk.java.net/~ppunegov/8156852/webrev.01/>

— Thanks,
Pavel Punegov

> On 15 Aug 2016, at 20:40, Kirill Zhaldybin <kirill.zhaldybin at oracle.com> wrote:
> 
> Pavel,
> 
> A bit of nitpicking:
> 
> 1. Could you please renew year in copyright in http://cr.openjdk.java.net/~ppunegov/8156852/webrev/raw_files/new/src/share/vm/utilities/json.cpp <http://cr.openjdk.java.net/~ppunegov/8156852/webrev/raw_files/new/src/share/vm/utilities/json.cpp> ?
> 2. Some lines http://cr.openjdk.java.net/~ppunegov/8156852/webrev/test/native/utilities/test_json.cpp.html <http://cr.openjdk.java.net/~ppunegov/8156852/webrev/test/native/utilities/test_json.cpp.html> are too long. Could you please re-format them?
> 
> Thank you.
> 
> Regards, Kirill
> 
> On 15.08.2016 20:27, Pavel Punegov wrote:
>> Please review this fix converting JSON_test from internalVMTests to Gtest.
>> All test cases in JSONTest::test() converted to Gtest’s TEST.
>> Replaced a UTF symbols to appropriate hex-representations.
>> 
>> bug: https://bugs.openjdk.java.net/browse/JDK-8156852
>>  <https://bugs.openjdk.java.net/browse/JDK-8156852>webrev: http://cr.openjdk.java.net/~ppunegov/8156852/webrev/ <http://cr.openjdk.java.net/%7Eppunegov/8156852/webrev/>
>> — Thanks,
>> Pavel Punegov
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160816/f8daba21/attachment.html>