RFR 8010325 : Remove hash32() method and hash32 int field from java.lang.String

Brent Christian brent.christian at oracle.com
Thu Jun 13 19:46:56 UTC 2013


On 6/12/13 7:55 PM, David Holmes wrote:
> Something of an aside but ...
>
> On 13/06/2013 3:45 AM, Martin Buchholz wrote:
>> Hi Brent,
>>
>> Thanks for doing this.
>>
>> Your webrev does not include mercurial changeset information, which I
>> think
>> is supported by recent webrevs.
>
> Given the changeset has to be created after the review is complete
> most/many people will not have a changeset prepared at review time.

That's it exactly.  If at all possible, I don't commit until the code 
has completed code review.  I tell webrev to do its thing based on 
modified files, rather than outgoing changesets.

-Brent



More information about the core-libs-dev mailing list