Closed Bug 425080 Opened 12 years ago Closed 12 years ago

C++ compatibilty: jsdbgapi.cpp: use of JS_malloc needs a cast

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9

People

(Reporter: jorendorff, Assigned: jorendorff)

Details

Attachments

(1 file)

Attached patch v1Splinter Review
The patch says it all:
-                code = JS_malloc(cx, script->length * sizeof(jsbytecode));
+                code = (jsbytecode *)
+                       JS_malloc(cx, script->length * sizeof(jsbytecode));
Attachment #311669 - Flags: review?(brendan)
Comment on attachment 311669 [details] [diff] [review]
v1

Sorry, I should know better by now.

/be
Attachment #311669 - Flags: review?(brendan)
Attachment #311669 - Flags: review+
Attachment #311669 - Flags: approval1.9b5?
Comment on attachment 311669 [details] [diff] [review]
v1

beta 5 is closed, moving approval flag ...
Attachment #311669 - Flags: approval1.9b5? → approval1.9?
Assignee: general → jorendorff
Comment on attachment 311669 [details] [diff] [review]
v1

a=beltzner
Attachment #311669 - Flags: approval1.9? → approval1.9+
Checking in js/src/jsdbgapi.c;
/cvsroot/mozilla/js/src/jsdbgapi.c,v  <--  jsdbgapi.c
new revision: 3.143; previous revision: 3.142
done
Status: NEW → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9
Flags: in-testsuite-
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.