Closed Bug 1162335 Opened 5 years ago Closed 3 years ago

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

Categories

(Core :: MFBT, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox40 --- affected
firefox55 --- fixed

People

(Reporter: jld, Assigned: xidorn)

References

Details

Attachments

(1 file)

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: 3 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.