Closed Bug 1116906 Opened 6 years ago Closed 6 years ago

Move AlreadyAddRefed.h to MFBT

Categories

(Core :: MFBT, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: ehsan, Assigned: ehsan)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Attached patch Move AlreadyAddRefed.h to MFBT (obsolete) — Splinter Review
Attachment #8543120 - Flags: review?(nfroyd)
Assignee: nobody → ehsan
Blocks: 1115033
Comment on attachment 8543120 [details] [diff] [review]
Move AlreadyAddRefed.h to MFBT

Review of attachment 8543120 [details] [diff] [review]:
-----------------------------------------------------------------

This patch is fine as far as it goes, but if this is going into MFBT, the comments in AlreadyAddRefed.h need some cleanup:

- Need to explain why already_AddRefed isn't in mozilla::
- Need to explain where these weird nsRefPtr/nsCOMPtr classes come from
- Maybe partially address the above by rewriting things to refer to generic ref-counted smart pointer classes where possible (e.g. the class comment could be "already_AddRefed cooperaters (sic) with refcounted smart pointers to enable you to assign...")
- The comment for downcast() needs to ditch the bit about dont_AddRef, since that's XPCOMish
Attachment #8543120 - Flags: review?(nfroyd) → feedback+
Attachment #8543120 - Attachment is obsolete: true
Attachment #8544068 - Flags: review?(nfroyd)
Attachment #8544068 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/a194a9a118f1
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.