Review request for 6843003: Windows Server 2008 R2 system recognition
Mark Reinhold
mr at sun.com
Tue Jun 30 00:08:37 UTC 2009
> Date: Mon, 29 Jun 2009 21:39:58 +0100
> From: alan.bateman at sun.com
> Xueming Shen wrote:
>> ...
>>
>> It might be better to simply initialize the sprops.os_name to
>> "Windows NT (unknown)" at
>> the very beginning of the nt block and then depends on the version
>> number to see if we have
>> a better guess.
>>
>> (2)We have been doing the same exercise every time MS released a new
>> os, it might be the
>> time to extract this out to a "configurable" file and simple do a
>> lookup based on the platform,
>> verMajor, verMinor.
>
> It is true that we need to touch this code whenever a new edition of
> Windows comes along. As it runs at startup I would be hesitant to
> introduce a lookup file.
Indeed. We should be reducing the number of files we open at startup,
not adding to them.
- Mark
More information about the core-libs-dev
mailing list