RFR: 8067306: Improve STATIC_ASSERT
Kim Barrett
kim.barrett at oracle.com
Fri Dec 12 17:12:51 UTC 2014
Thanks for looking at this.
On Dec 12, 2014, at 3:27 AM, Erik Helin <erik.helin at oracle.com> wrote:
>
> On 2014-12-12, Kim Barrett wrote:
>> Please review this enhancement to the hotspot STATIC_ASSERT() macro.
>>
>> […]
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8067306
>>
>> Webrev:
>> http://cr.openjdk.java.net/~kbarrett/8067306/webrev/
>
> Looks good, thanks for improving this. Do we really need the
> implementation note? The implementation itself is pretty clear to me.
The bit about dependent types isn't entirely obvious and is easy to
forget; I was worried someone might later wonder why it looks more
complicated than seems necessary and "simplify" it. (I haven't
audited current uses to see if that would immediately fail.) But I'm
fine with removing it.
More information about the hotspot-runtime-dev
mailing list