Looks good! Volker
On Mon, Dec 19, 2016 at 2:33 PM, Thomas Stüfe <thomas.stu...@gmail.com> wrote: > > On Sun, Dec 18, 2016 at 1:13 PM, Thomas Stüfe <thomas.stu...@gmail.com> > wrote: >> >> HI all, >> >> after investigating I see a number of issues preventing gtest from running >> on AIX, so I decided to get the fix for the wrong assert in the hotspot out >> of the way and open follow up issues for the other problems. >> >> So, this is just the fixed assert, fixed in the fashion David suggested. >> All other issues I will fix separately later. >> >> >> http://cr.openjdk.java.net/~stuefe/webrevs/8171225-aix-fix-gtest-assert-compile-error/webrev.01 >> >> (Hope the link is correct, the website seems down at the moment). >> >> Thank you for reviewing, >> >> Thomas > > > (openjdk.java.net is still down, so I attach here the webrev zip) >