<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="auto">Looks good and trivial.</div><div dir="auto"><br></div><div dir="auto">I remember running into this before, see 8171225. What I do not understand is, should NULL not be (void*)0 according to Posix: <a href="http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/stddef.h.html">http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/stddef.h.html</a> ? So this should be a pointer-pointer comparison.</div><div dir="auto"><br></div><div dir="auto">Maybe we just miss include <stddef.h> somehow on AIX?<br><div dir="auto"><br></div><div dir="auto">.. Thomas </div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 20, 2018, 17:03 Volker Simonis <<a href="mailto:volker.simonis@gmail.com" target="_blank">volker.simonis@gmail.com</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
can I please have a review fort he following trivial gtest fix which<br>
currently breaks the AIX build:<br>
<br>
<a href="http://cr.openjdk.java.net/~simonis/webrevs/2018/8214125/" rel="noreferrer noreferrer" target="_blank">http://cr.openjdk.java.net/~simonis/webrevs/2018/8214125/</a><br>
<a href="https://bugs.openjdk.java.net/browse/JDK-8214125" rel="noreferrer noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8214125</a><br>
<br>
Change JDK-8213352 introduced an gtest which does a comparison between<br>
an integer and a pointer which is forbidden in C++:<br>
<br>
    ASSERT_EQ(NULL, nodes[i]->next());<br>
<br>
I don't know why other compilers havn't complained about it, but XLC did.<br>
<br>
The fix for this is trivial - just cast NULL to the appropriate pointer type:<br>
<br>
    ASSERT_EQ((BufferNode*)NULL, nodes[i]->next());<br>
<br>
<br>
Thank you and best regards,<br>
Volker<br>
</blockquote></div>