mozilla::AutoTimelineMarker should delte copy and move ctors and assignment

RESOLVED FIXED in Firefox 40

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: fitzgen, Assigned: fitzgen)

Tracking

unspecified
mozilla40
Points:
---

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment)

It should really stay where it is, we don't want to accidentally add duplicate starts/ends or something.
Comment on attachment 8599557 [details] [diff] [review]
mozilla::AutoTimelineMarker shouldn't be copy-able

So copy ctor and assignment operator, not move ctor
Attachment #8599557 - Flags: review?(bugs) → review+
(In reply to Olli Pettay [:smaug] from comment #2)
> Comment on attachment 8599557 [details] [diff] [review]
> mozilla::AutoTimelineMarker shouldn't be copy-able
> 
> So copy ctor and assignment operator, not move ctor

Yup. Implicit move constructor/assignment won't be generated by the compiler because there is a user defined destructor.
https://hg.mozilla.org/mozilla-central/rev/4620352fe92e
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.