RFR: JI-29: SA ClassDump throws exception after permgen removal

[Yumin Qi]

Hi, All

   Based on Coleen's feedback, sadis.c has compilation warning, fixed, 
also removed some unused code in SA.

http://cr.openjdk.java.net/~minqi/8000332/ 
<http://cr.openjdk.java.net/%7Eminqi/8000332/>

Thanks
Yumin

On 10/2/2012 1:50 PM, Yumin Qi wrote:
> Hi, all
>
>   This is real URL for this bug codereview, I accidentally created the 
> bug with JIRA by click incidents link.  Now the link is
> http://cr.openjdk.java.net/~minqi/JDK-8000332/ 
> <http://cr.openjdk.java.net/%7Eminqi/JDK-8000332/>
>
>   Thanks
>   Yumin
>
> On 10/2/2012 10:33 AM, Yumin Qi wrote:
>> Hi, May I have your codereview
>>
>> http://cr.openjdk.java.net/~minqi/ji-29 
>> <http://cr.openjdk.java.net/%7Eminqi/ji-29/>
>>
>> JI-29: SA ClassDump throws exception after permgen removal
>> Summary: In ClassWrite.writeFields(), fields count was set to fields 
>> length which overflows the array index.
>>
>> Also removed a file which should be removed in changeset of 6879063: 
>> SA should use hsdis.
>>
>> Reviewed-by:
>> Contributed-by: yumin.qi at oracle.com
>>
>>
>> Thanks
>> Yumin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20121002/f00b258a/attachment.html