RFR: 8067306: Improve STATIC_ASSERT
jon.masamitsu at oracle.com
Mon Dec 15 18:49:10 UTC 2014
On 12/12/2014 09:12 AM, Kim Barrett wrote:
> 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.
>> 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.
I like the comment. I'd vote to keep it.
More information about the hotspot-gc-dev