Closed Bug 1168179 Opened 4 years ago Closed 4 years ago
Pres Context::m Medium as MOZ _UNSAFE _REF
No description provided.
Attachment #8610206 - Flags: review?(dbaron)
I guess I don't really understand (from the comments in Attributes.h) how MOZ_UNSAFE_REF differs from MOZ_NON_OWNING_REF (or, for that matter, MOZ_OWNING_REF, which it appears to actually be more similar to).
MOZ_NON_OWNING_REF is referring to the pattern where a raw pointer is set up such that before the value it is pointing to is invalidated, the pointer has been nulled out. This is often done in the destructor of the pointed-at value. MOZ_OWNING_REF is referring to the pattern where manual AddRefs and Releases are used to manage the lifetime of an object. Something which is MOZ_OWNING_REF would usually be a smart pointer, except that there are reasons why it cannot be one (for example, a MOZ_OWNING_REF could exist inside of a union, or in another place where a POD object is required). MOZ_UNSAFE_REF is for everything else which needs to be handled. For example, in this situation mMedium is MOZ_UNSAFE_REF because it is never AddRef/Released, nor is it nulled out before it's target becomes invalid, but rather it's target is guaranteed to always be valid because it is a static atom.
Comment on attachment 8610206 [details] [diff] [review] Mark nsPresContext::mMedium as MOZ_UNSAFE_REF Please leave the comment "//initialized by subclass ctors". You could also shorten it to: MOZ_UNSAFE_REF("always a static atom") ("mMedium is only ever assigned" doesn't really add anything, and the example might not remain true in the future) Please include the bug number in the commit message. r=dbaron with that
Attachment #8610206 - Flags: review?(dbaron) → review+
You need to log in before you can comment on or make changes to this bug.