Request for review (XS): 8010116: Abstract_VM_Version::internal_vm_info_string() should recognize VS2010 and VS2012 (original) (raw)
Zhengyu Gu zhengyu.gu at oracle.com
Thu Mar 14 13:10:25 PDT 2013
- Previous message: Request for review (XS): 8010116: Abstract_VM_Version::internal_vm_info_string() should recognize VS2010 and VS2012
- Next message: Request for review (XS): 8010116: Abstract_VM_Version::internal_vm_info_string() should recognize VS2010 and VS2012
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Looks good to me.
-Zhengyu
On 3/14/2013 3:44 PM, Krystal Mo wrote:
Hi all,
Could I get a review for this small fix, please? Webrev: http://cr.openjdk.java.net/~kmo/8010116/webrev.00/ CR: http://bugs.sun.com/viewbug.do?bugid=8010116 Description: Currently AbstractVMVersion::internalvminfostring() recognizes MSVC versions from 5.0 to 9.0 (VS2008). Since JDK7 can be and is being built with VS2010, it should recognize that compiler (along with new VS2012), so that when its value appears in crash logs, it shouldn't say: "unknown MS VC++:1600", which is what can be seen in JDK7u's crash log. Thanks, Kris
- Previous message: Request for review (XS): 8010116: Abstract_VM_Version::internal_vm_info_string() should recognize VS2010 and VS2012
- Next message: Request for review (XS): 8010116: Abstract_VM_Version::internal_vm_info_string() should recognize VS2010 and VS2012
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]