'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