Closed Bug 1184564 Opened 5 years ago Closed 5 years ago

Remove jsval

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: jandem, Assigned: jandem)

Details

(Keywords: dev-doc-complete)

Attachments

(5 files)

jsval is a typedef for JS::Value so this is pretty straight-forward.
Attachment #8634728 - Flags: review?(bobbyholley)
This patch is surprisingly small but it's all we need to build, so..
Attachment #8634730 - Flags: review?(bzbarsky)
Attached patch Part 3 - CTypesSplinter Review
Attachment #8634732 - Flags: review?(arai.unmht)
Attached patch Part 4 - JSSplinter Review
Attachment #8634733 - Flags: review?(evilpies)
Attachment #8634734 - Flags: review?(jwalden+bmo)
Comment on attachment 8634730 [details] [diff] [review]
Part 2 - dom/ and storage/

r=me
Attachment #8634730 - Flags: review?(bzbarsky) → review+
Attachment #8634728 - Flags: review?(bobbyholley) → review+
Attachment #8634734 - Flags: review?(jwalden+bmo) → review+
Comment on attachment 8634732 [details] [diff] [review]
Part 3 - CTypes

Review of attachment 8634732 [details] [diff] [review]:
-----------------------------------------------------------------

https://dxr.mozilla.org/mozilla-central/source/js/src/ctypes/CTypes.cpp#6379
>   AutoValueVector argTypes(cx);
>   RootedObject arrayObj(cx, nullptr);
> 
>   if (args.length() == 3) {
>     // Prepare an array of jsvals for the arguments.

As other comments are fixed, would you change "jsvals" to "Values" here for consistency?
Attachment #8634732 - Flags: review?(arai.unmht) → review+
Attachment #8634733 - Flags: review?(evilpies) → review+
Keywords: leave-open
Keywords: leave-open
(In reply to Florian Scholz [:fscholz] (MDN) from comment #10)
> arai updated:

Thanks arai. Much appreciated.
You need to log in before you can comment on or make changes to this bug.