'ciConstant' and 'ciField' unnecessarily declare 'print_impl()' method
Tom Rodriguez
tom.rodriguez at oracle.com
Fri Dec 10 09:46:32 PST 2010
On Dec 10, 2010, at 6:44 AM, Volker Simonis wrote:
> Hi,
>
> while debugging I just realized that the two classes 'ciConstant' and
> 'ciField' unnecessarily declare a 'print_impl()' method which is never
> defined nor called. They have probably been declared there because the
> two class names suggest that the classes are derived from 'ciObject'
> but that's not the case.
>
> To avoid further confusion I'd suggest to simply remove the two
> declarations. If somebody will open a bug I'll volunteer to submit the
> patch.
> But perhaps it's easier to remove the two useless declarations as part
> of another change to save bug IDs:)
I've been accumulating some dead code changes so I'll add these to it. Any other dead things or thing that you suspect are dead?
tom
>
> Regards,
> Volker
More information about the hotspot-compiler-dev
mailing list