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

[Daniel Fuchs]

On 28/04/15 16:54, Jonathan Gibbons wrote:
> I'll sponsor the patch for you.

Oh - the version needs to be updated too. Sorry I didn't catch that before.

cheers,

-- daniel

>
> -- Jon
>
> On 04/28/2015 06:56 AM, Jesper Wilhelmsson wrote:
>> 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
>>>
>