RFR: 8068396: Rename assert() to vmassert()
kim.barrett at oracle.com
Mon Jan 12 15:58:15 UTC 2015
On Jan 12, 2015, at 4:02 AM, Erik Helin <erik.helin at oracle.com> wrote:
> On 2015-01-12, David Holmes wrote:
>> On 10/01/2015 4:11 AM, Kim Barrett wrote:
>>> New webrev: http://cr.openjdk.java.net/~kbarrett/8068396/webrev.03/
>>> Incremental webrev: http://cr.openjdk.java.net/~kbarrett/8068396/webrev.03.incr/
>> Webrev looks fine but for some reason you patch file did not update:
>> still seems to be version 1.
> I suspect that Kim is using `webrev -r` to create the webrev, this is
> known to produce broken patch files:
Using “webrev -r” is indeed what I’m doing. I was unaware of that bug.
Any suggestions for what else I should do? I’m using hg mq’s to manage the changes locally.
That seems to require using -r when creating the webrev; “-r qparent” for the full webrev, and
“-r <prev-mq-patch>” for the incremental webrev.
More information about the hotspot-dev