Closed Bug 1439063 Opened 7 years ago Closed 7 years ago

Move a few public headers from js/src into js/public

Categories

(Core :: JavaScript Engine, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: jorendorff, Assigned: jorendorff)

References

Details

Attachments

(3 files)

Affected: jsalloc.h, jsbytecode.h, jsprf.h, jsprototypes.h, jsversion.h, jswrapper.h Also possibly jstypes.h and jspubtd.h. I will not mess with jsapi.h and jsfriendapi.h.
Depends on: 1440043
Assignee: nobody → jorendorff
Status: NEW → ASSIGNED
js/src/jsalloc.h -> js/public/AllocPolicy.h jsalloc.cpp -> js/src/util/AllocPolicy.cpp jsbytecode.h -> merge into js/public/TypeDecls.h jsprf.h -> js/public/Printf.h jsprf.cpp -> js/src/util/Printf.cpp jsprototypes.h -> public/ProtoKey.h jswrapper.h -> js/Wrapper.h
Attachment #8953220 - Flags: review?(jdemooij)
Attachment #8953221 - Flags: review?(jdemooij)
Comment on attachment 8953220 [details] [diff] [review] Part 1: Move several public headers from js/src to js/public Review of attachment 8953220 [details] [diff] [review]: ----------------------------------------------------------------- So much nicer.
Attachment #8953220 - Flags: review?(jdemooij) → review+
Attachment #8953221 - Flags: review?(jdemooij) → review+
Comment on attachment 8953219 [details] [diff] [review] Part 0: BitArray.h uses memset and therefore must #include <string.h> Review of attachment 8953219 [details] [diff] [review]: ----------------------------------------------------------------- Oops forgot part 0.
Attachment #8953219 - Flags: review?(jdemooij) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/7089df4b037451cd7cfa6176a0011ec50048d91c Bug 1439063 - Part 0: BitArray.h uses memset and therefore must #include <string.h>. r=jandem. https://hg.mozilla.org/integration/mozilla-inbound/rev/0ceb91c42b0ff876a61ed23f5e587ba3a2083bcf Bug 1439063 - Part 1: Move several public headers from js/src to js/public. r=jandem. https://hg.mozilla.org/integration/mozilla-inbound/rev/51fe9a44a5d3f1448cd7a2e50077e80ef919a3f3 Bug 1439063 - Part 2: Remove some unnecessary uses of js/Printf.h. r=jandem.
Depends on: 1441435
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: