RFR: CODETOOLS-7901397 - Flag -b is ignored in webrev

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Tue Apr 28 13:56:49 UTC 2015


Thanks for reviewing!

I will also need a sponsor since I'm not a committer in the Code-tools project.

Thanks,
/Jesper


Daniel D. Daugherty skrev den 28/4/15 15:48:
>  > Webrev: http://cr.openjdk.java.net/~jwilhelm/CT-7901397/webrev.00/
>
> webrev.ksh
>      No comments.
>
>
> I have a similar change to a much older version of webrev that I've
> been fixing relative to /java/devtools/share/bin/webrev so this
> looks good to me.
>
> I'll have to check out that project since I've done webrev fixes
> in the past.
>
> Dan
>
>
>
>
> On 4/28/15 6:52 AM, Jesper Wilhelmsson wrote:
>> Widening the audience since I didn't get any replies on webrev-dev. Is anyone
>> reading that list?
>>
>> 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