Closed Bug 1856746 Opened 2 years ago Closed 2 years ago

Update already_AddRefed documentation to mention a case where it makes sense as a parameter

Categories

(Core :: MFBT, task)

Product:
Core
Component:
MFBT
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
120 Branch
Tracking Flags:
Tracking Status
firefox120 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(1 file)

Bug 1856746: Extend already_AddRefed documentation to mention a case where it makes sense as a function parameter. r?nika
48 bytes, text/x-phabricator-request
Details | Review

Per discussion in slack today, there are cases where already_AddRefed makes sense as a function-param. Right now, its initial paragraph of documentation gives the impression that it only makes sense as a return value; let's extend that a little bit to mention a case where it's a reasonable choice as a function-parameter.

Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Pushed by dholbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4f4d4deb2aab Extend already_AddRefed documentation to mention a case where it makes sense as a function parameter. r=nika

https://hg.mozilla.org/mozilla-central/rev/4f4d4deb2aab

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox120: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 120 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: