So yes .. it changes that .. but could you explain why the new string is correct ? I don't see an evaluation here, or in the bug report - where there should be one!
Also the indentation seems wrong  - even when I look at the "raw" file.

-phil.

On 2/8/19 1:07 AM, Shashidhara Veerabhadraiah wrote:

Hi All, Please review a fix for the below customer bug.

Bug: https://bugs.openjdk.java.net/browse/JDK-8190361

Webrev: http://cr.openjdk.java.net/~sveerabhadra/8190361/webrev.01/

The version string for the jaccessinspector and jaccesswalker modules had the string “JDK_BUILD_ID” and this change fixes that. The only doubt I had is that, hopefully these resource files are not generated from a different file and is editable by manual way.

Thanks and regards,

Shashi


Reply via email to