Request for review : 7088913 : (original) (raw)
David Holmes david.holmes at oracle.com
Thu Oct 25 04🔞26 UTC 2012
- Previous message: Request for review : 7088913 :
- Next message: Request for review : 7088913 :
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Mike,
Looks okay to me. I assume the ccc request will be resuscitated :)
David
On 25/10/2012 2:07 PM, Mike Duigou wrote:
A long, long time ago (2011) there was a review request[1] for
CR#7088913 : Add compatible static hashCode(primitive) to primitive wrapper classes The change was approved by the three cc:ed reviewers. At that time the webrev was co-mingled with another changeset that had more controversial changes. Neither changeset was committed and 7088913 got ignored and eventually forgotten. Until now. Since so much time has passed I am asking for review again so that the commit is not a surprise to anyone. The webrev incorporates Joe's suggestions from long ago and now including an expanded (though probably not complete) unit test. http://cr.openjdk.java.net/~mduigou/7088913/1/webrev/ Mike [1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2011-September/007733.html
- Previous message: Request for review : 7088913 :
- Next message: Request for review : 7088913 :
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]