[8u20] Request for approval: 8033798 - javap output has unnecessary trailing whitespace

[Alexander Zuev]

Jeff,

   yes, the fix fixed two additional issues.

/Alex

On 2/11/14 18:53, Jeff Dinkins wrote:
> Hi Alex,
>
> Thanks for the webrev :)
>
> Looks good - approved for 8u-dev.
>
> Just to be clear, this also *fixes* 8033581 & 8033726, right?
>
> -jeff
>
>
> On Feb 11, 2014, at 8:41 AM, Alexander Zuev <alexander.zuev at oracle.com> wrote:
>> Take two:
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8033798
>> Fix pushed as: http://hg.openjdk.java.net/jdk9/dev/langtools/rev/b5c6e4558b06
>> Fix applies without an issue to http://hg.openjdk.java.net/jdk8u/jdk8u-dev/langtools
>> Webrev is available at http://cr.openjdk.java.net/~kizune/8033968/webrev
>>   This fix also eliminates issues:
>> JDK-8033581: Incorrect comment aligment (https://bugs.openjdk.java.net/browse/JDK-8033581)
>> and
>> JDK- 8033726: StackMapTable does not unindent properly (https://bugs.openjdk.java.net/browse/JDK-8033726)
>>
>> /Alex
>>