RFR: 8067306: Improve STATIC_ASSERT

Coleen Phillimore coleen.phillimore at oracle.com
Fri Jan 9 13:45:30 UTC 2015


This looks like a good change.   I didn't realize we had 
STATIC_ASSERT.   Is there any supporting opinion to make it lower case 
so it doesn't scream at you in the code?

Coleen

On 1/9/15, 4:00 AM, Erik Helin wrote:
> On 2015-01-08, Kim Barrett wrote:
>> On Jan 7, 2015, at 7:25 AM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
>>> I'll sponsor and review the patch if you change "(Cond) ? true : false" to just "(Cond)". Any potential problems with ancient compilers can be dealt with when / if they are causing problems.
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8067306
>> New full webrev: http://cr.openjdk.java.net/~kbarrett/8067306/webrev.03/
>> Incremental webrev: http://cr.openjdk.java.net/~kbarrett/8067306/webrev.03.incr/
> Still good, Reviewed.
>
> Thanks,
> Erik



More information about the hotspot-runtime-dev mailing list