RFR 8139163: InstanceKlass::cast passes through NULL (original) (raw)

Kim Barrett kim.barrett at oracle.com
Fri Oct 23 22:44:50 UTC 2015


On Oct 23, 2015, at 5:34 PM, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:

On 10/23/15 4:30 PM, Kim Barrett wrote: src/share/vm/oops/klass.hpp

Pre-existing: Why aren't Klass::vtable and Klass::vtablelength pure virtual? I don't know! yes, they should be and it feels a lot safer to have them pure virtual so nobody gets null/0 by mistake.

Feel free to slip that change in. Or file an RFE if not comfortable with including here.

Do you want to see another webrev with the additional cleanups?

Yes, please. They’re pretty straightforward, but there were enough of them…



More information about the hotspot-dev mailing list