'ciConstant' and 'ciField' unnecessarily declare 'print_impl()' method

Volker Simonis volker.simonis at gmail.com
Fri Dec 10 06:44:22 PST 2010


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:)

Regards,
Volker


More information about the hotspot-compiler-dev mailing list