On 11/10/2012 11:42, Alan Bateman wrote:
On 10/10/2012 21:18, Dan Xu wrote:
Thanks for your good comments.
I have changed access modifiers for methods in WinNTFileSystem.java.
And the new webrev can be viewed at
http://cr.openjdk.java.net/~dxu/7186817/webrev.01/
I did not change the hashCode implementation in this version. It will
need further complete tests and another round of code review. But it
is a good point to improve our code performance. Please log another
bug to address this issue. Thanks!
-Dan
Thanks Dan, it looks good to me. I'll push this shortly for you.
Dan - I've pushed this for you but without the change to the following file:
make/tools/sharing/classlist.windows
The reason is that file is auto-generated and has a hash at the end so
it's not meant to be manually edited. The class list is used by
-Xshare:dump to generate the shared archive for class data sharing and
is tolerant of missing classes. I think the class lists are a bit of
date anyway and are overdue a refresh.
-Alan
