Closed Bug 1065774 Opened 7 years ago Closed 7 years ago

Be more precise in PromiseFlatString docs about why assigning PromiseFlatString(...) to a reference is safe

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla35

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(1 file)

I didn't know assigning a tepmorary to a referenc, extended the temporary's lifetime to that of the reference.  It would be nice if the in-file comment were more precise about why this is happening, citing chapter/verse for authority.
Attached patch PatchSplinter Review
Attachment #8487611 - Flags: review?(nfroyd)
Attachment #8487611 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/5c03643c3180
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.