RFR (S): Recognize windows 8

Mikael Vidstedt mikael.vidstedt at oracle.com
Fri Jun 1 10:20:20 PDT 2012


I fully agree with you that that is a good idea and I actually started 
doing that first, I still have that change in another sandbox. It became 
rather big and scary in the end though so I'm proposing that I do that 
as a separate change after 7u6 to reduce the risk. Ok?

Cheers,
Mikael

On 2012-06-01 07:32, Dmitry Samersoff wrote:
> Mikael,
>
> Is it possible to refactor this code to get rid of
> multiple
>
> if (si.wProcessorArchitecture == PROCESSOR_ARCHITECTURE_AMD64)
>                st->print(" , 64 bit");
>
> lines.
>
> -Dmitry
>
> On 2012-05-31 19:28, Mikael Vidstedt wrote:
>> This change adds support for recognizing Windows 8 in
>> os::print_windows_version, inspired by the corresponding changes in the
>> libraries [1][2].
>>
>> Verified on a Windows 8 machine and passes JPRT.
>>
>> http://cr.openjdk.java.net/~mikael/7170275/webrev.00
>>
>> Please help me review, and I'll also be needing a sponsor.
>>
>> Thanks,
>> Mikael
>>
>> [1] http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7170169
>> [2] http://cr.openjdk.java.net/~khazra/7170169/7u6/webrev.00
>>
>



More information about the hotspot-runtime-dev mailing list