This is a nice cleanup. Just a couple of comments:
- Update copyrights now that it is 2019
- For the test, is the source code always guaranteed to be there? I was
not sure if that was a requirement. Or does the test still pass if it
can't find the source code?
Thanks,
Sean
On 12/27/18 3:11 AM, Weijun Wang wrote:
Please take a review at
https://cr.openjdk.java.net/~weijun/8215937/webrev.00/
A new Usages.java test is added to make sure the strings in various
Resources.java files are exactly what are used in security-related source
files, no more no less.
Two old tests are removed. NewNamesFormat.java checks for format and
Usages.java covers it. NewResourcesNames.java is a manual test and is too stale
and not easy to run.
Several calls in keytool and jarsigner are modified to follow a more consistent calling
convention (always rb.getString(string_literal)) so that they can be detected by
Usages.java more easily. There are still several places in PolicyFile.java calling
'LocalizedMessage.getNonlocalized(POLICY + "...", source)' but I left them
unchanged and dealt with it specially in Usages.java.
Many useless strings in Resources.java files are removed. I've double checked
each and made sure the related calls were removed some time ago.
Thanks
Max
