auto merge of #5015 : Ms2ger/servo/finalize, r=jdm

Previously, we had 'value' and 'this' locals, both storing a pointer to the
DOM object, for no good reason.
This commit is contained in:
bors-servo 2015-02-22 08:36:49 -07:00
commit 92620c0a5f

View file

@ -4040,10 +4040,9 @@ let this: *const %s = unwrap::<%s>(obj);
def finalizeHook(descriptor, hookName, context):
release = """\
let value = unwrap::<%s>(obj);
let _ = Box::from_raw(value as *mut %s);
let _ = Box::from_raw(this as *mut %s);
debug!("%s finalize: {:p}", this);\
""" % (descriptor.concreteType, descriptor.concreteType, descriptor.concreteType)
""" % (descriptor.concreteType, descriptor.concreteType)
return release
class CGClassTraceHook(CGAbstractClassHook):