RFR: 8067306: Improve STATIC_ASSERT

[Coleen Phillimore]

On 1/9/15, 8:45 AM, Coleen Phillimore wrote:
>
> 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?

Never mind. It's very macro.  It should probably stay uppercase.

Coleen

>
> 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
>