RFR: CODETOOLS-7901397 - Flag -b is ignored in webrev
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