Hi,
I don't think another review is needed, but FWIW this looks good to
me!
/Claes
On 2019-12-20 11:03, Baesken, Matthias wrote:
Hi Erik, thanks for the review ! I'll remove the comment line .
May I get a second review ?
Best regards, Matthias
Hello Matthias,
Looks good except for the comment. There is no need to add a comment for
something that has been removed, it will just look confusing.
Documenting it in the JBS issue is enough.
Thanks for cleaning this up!
/Erik
On 2019-12-20 09:37, Baesken, Matthias wrote:
Hello, please review this small MSVC related change .
The MSVC based builds still have the old flag -d2Zi+ set; this is an
undocumented flag , the name of the flag for enhanced optimized
debugging is since VS2013 -Zo .
However the flag (-Zo) is enabled by default for a long time , so the old
undocumented version of it ("-d2Zi+") should be removed completely.
See the VS2017 documentation :
https://docs.microsoft.com/en-us/cpp/build/reference/zo-enhance-
optimized-debugging?view=vs-2017
"The /Zo option is enabled by default in Visual Studio when you specify
debugging information with /Zi or /Z7. Specify /Zo- to explicitly disable this
compiler option.
The /Zo switch is available starting in Visual Studio 2013 Update 3, and it
replaces the previously undocumented /d2Zi+ switch."
Bug/webrev :
https://bugs.openjdk.java.net/browse/JDK-8236274
http://cr.openjdk.java.net/~mbaesken/webrevs/8236274.0/
Thanks, Matthias