RFR: 8161145: The min/max macros make hotspot tests fail to build with GCC 6

[Kim Barrett]

> On Jun 1, 2017, at 5:49 AM, Per Liden <per.liden at oracle.com> wrote:
>> 
>> FWIW, another option would be to remove all the min/max stuff here,
>> and add -DNOMINMAX (or whatever the proper syntax is) in the Windows
>> build configuration.  I did some later research on JDK-8161145, and
>> defining that macro seems to be the "official" way to suppress those
>> macros in the offending windows header.
> 
> We have a similar problem with min/max on Solaris, so it's not only a Windows issue. The approach Erik's patch takes seems more portable.

Oh, I didn’t know Solaris has this bug too.  I retract the -DNOMINMAX suggestion.  Go with the blue paint approach.

> cheers,
> Per
> 
>> 
>> The proposed change also seems fine to me, not surprisingly.  Under the
>> circumstances, not sure I should be counted as a reviewer :)
>> 
>> I can sponsor though.