Remove already_AddRefed::MatchNullptr, now that we can use nullptr

RESOLVED FIXED in Firefox 55

Status

()

defect
RESOLVED FIXED
4 years ago
Last year

People

(Reporter: jld, Assigned: xidorn)

Tracking

Trunk
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox40 affected, firefox55 fixed)

Details

Attachments

(1 attachment)

mfbt/AlreadyAddRefed.h says:

   * We can revert to simply explicit (T*) and implicit (decltype(nullptr)) when
   * nullptr no longer needs to be emulated to support the ancient b2g compiler.

This is now the case, so that can be done.  I didn't see an existing bug for this, so I'm filing one.
Assignee: nobody → xidorn+moz
Blocks: 1365254
Comment on attachment 8873303 [details]
Bug 1162335 - Remove already_AddRefed::MatchNullptr.

https://reviewboard.mozilla.org/r/144752/#review148754
Attachment #8873303 - Flags: review?(nfroyd) → review+
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4a2b5682bddf
Remove already_AddRefed::MatchNullptr. r=froydnj
https://hg.mozilla.org/mozilla-central/rev/4a2b5682bddf
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Depends on: 1470641
You need to log in before you can comment on or make changes to this bug.