RFR(XS): Remove unused ShenandoahPrinter

Zhengyu Gu zgu at redhat.com
Tue Aug 1 13:08:31 UTC 2017


Ah, okay then. Should we warp it around ASSERT

-Zhengyu

On 08/01/2017 09:03 AM, Aleksey Shipilev wrote:
> On 08/01/2017 02:59 PM, Zhengyu Gu wrote:
>> Removed no longer used ShenandoahPrinter
>>
>> http://cr.openjdk.java.net/~zgu/shenandoah/rm_sh_printer/webrev.00/
>
> Funny story about that: I had removed unused printing routines from ShenandoahHeap, but then
> Christine wanted to keep them around for debugging purposes. Thus ShenandoahPrinter was born, where
> these routines found their temporary home. And that kinda explains why it is unused. Talk to chf
> about that ;)
>
> Thanks,
> -Aleksey
>


More information about the shenandoah-dev mailing list