RFR(XXS) : 8166483 : gtest fmw should be updated to support null detection on SS >= 12u4

[Igor Ignatyev]

Kirill,

thank you for reviewing the patch.

Cheers,
— Igor

> On Sep 22, 2016, at 1:21 AM, Kirill Zhaldybin <kirill.zhaldybin at oracle.com> wrote:
> 
> Igor,
> 
> Looks good to me, thank you for fixing this.
> 
> Regards, Kirill
> 
> On 22.09.2016 00:43, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev/8166483/webrev.00/
>>> 3 lines changed: 1 ins; 0 del; 2 mod;
>> 
>> Hi all,
>> 
>> could you please review this tiny fix which enables null detection functionality in gtest on solaris if it’s been build by Solaris Studio >= 12.4? Due to disabled null detection, we have to explicitly cast NULL to a type of an actual value, i.e. instead of nice and clear 'ASSERT_EQ(NULL, myPointer)’ we have to write ‘ASSERT_EQ((MyClass*) NULL, myPointer)’. Reviewing 8165433[1], we came to the conclusion that ASSERT_EQ(NULL, myPointer) is what we really want. W/O this patch, the latest version of 8165433[2] fails during build on solaris.
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8166483
>> webrev: http://cr.openjdk.java.net/~iignatyev/8166483/webrev.00/
>> testing: I’ve run jprt w/ 8165433 integrated
>> 
>> [1] http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2016-September/020982.html
>> [2] http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8165433/webrev.03/
>> 
>> Thanks,
>> — Igor
>> 
>