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

Daniel Fuchs daniel.fuchs at oracle.com
Tue Apr 28 13:14:04 UTC 2015

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