mCX is unused in JetpackParent

RESOLVED FIXED

Status

Mozilla Labs
Jetpack Prototype
--
trivial
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

({coverity})

Trunk
coverity

Details

(Whiteboard: fixed-in-tracemonkey, URL)

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
/js/jetpack/JetpackParent.cpp (View Hg log or Hg annotations)
    * line 160 -- JSContext* mCX;

Found one matching line
(Assignee)

Comment 1

8 years ago
Created attachment 482494 [details] [diff] [review]
remove it
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #482494 - Flags: review?(anygregor)
Attachment #482494 - Flags: review?(anygregor) → review+
(Assignee)

Comment 2

7 years ago
anygregor: i don't know how you do pushes, could you please get whichever stamps are needed and get this pushed? thanks
Keywords: checkin-needed
No approval needed for tracemonkey (as long as the patch is sane).

Landed in TM: http://hg.mozilla.org/tracemonkey/rev/6a5b1f1a2296
Keywords: checkin-needed
Whiteboard: fixed-in-tracemonkey
(Assignee)

Comment 4

7 years ago
this was merged with https://hg.mozilla.org/mozilla-central/rev/dd753ec085e2
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.