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