Hi Mattias, the change looks good. Could you please also update the comment in the line above which still reads "STAT".
Also, maybe "STATI" would be a better name choice (longer is better :) because the probability of a clash is lower (and it would nicely align with QUEUE in the comments :) But I leave that up to you... Regards, Volker On Fri, Jun 15, 2018 at 9:47 AM, Baesken, Matthias <matthias.baes...@sap.com> wrote: > Please review this small change that fixes the AIX build after "8203641: > Refactor String Deduplication into shared" . > > We are getting this compilation error : > /build_ci_jdk_jdk_rs6000_64/src/hotspot/share/gc/shared/stringdedup/stringDedup.hpp", > line 107.38: 1540-0063 (S) The text "1" is unexpected. > > > Looks like the name of the second template parameter (STAT) > > template <typename QUEUE, typename STAT> > static void initialize_impl(); > > is clashing with defines from the AIX system headers (where I find #define > STAT 1 ) . > Renaming STAT to something else fixes the build on AIX . > > Webrev : > > http://cr.openjdk.java.net/~mbaesken/webrevs/8205091/ > > Bug : > > https://bugs.openjdk.java.net/browse/JDK-8205091 > > > Thanks, Matthias