RFR: CODETOOLS-7901397 - Flag -b is ignored in webrev (original) (raw)

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Tue Apr 28 13:36:15 UTC 2015

Thanks for the review! /Jesper

Daniel Fuchs skrev den 28/4/15 15:14:

On 28/04/15 14:52, Jesper Wilhelmsson wrote:

Widening the audience since I didn't get any replies on webrev-dev. Is anyone reading that list? Sorry Jesper, I intended to reply - but got distracted by other things :-( I have looked at your patch and it looks good to me. I have also used it for a while (without passing the -b flag however) - and my webrevs have been looking good to me. I am however not a Reviewer for CODE-TOOLS/webrev... best regards, -- daniel

Thanks, /Jesper

Jesper Wilhelmsson skrev den 20/4/15 17:04: Hi, Please review this fix to make webrev care about the -b flag and make it possible to produce diffs with whitespace changes. Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7901397 Webrev: http://cr.openjdk.java.net/~jwilhelm/CT-7901397/webrev.00/ I will also need a sponsor to push this change. Thanks, /Jesper

More information about the hotspot-dev mailing list