729382 - move the rest of the CallObject stuff into ScopeObject.cpp

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Luke Wagner [:luke]]

Attachment: mv

Missed this in the initial ScopeObject patch.  Keeps annoying me.  Patch just moves/renames stuff, no semantic changes.

Comment 1

[Jeff Walden [:Waldo]]

Comment on attachment 599440 [details] [diff] [review]
mv

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

I didn't look at the jsfun.cpp/ScopeObject.cpp bits, assuming they're entirely copypasta.

::: js/src/vm/ScopeObject.h
@@ +165,5 @@
>      inline void copyValues(uintN nargs, Value *argv, uintN nvars, Value *slots);
> +
> +    static JSBool getArgumentsOp(JSContext *cx, JSObject *obj, jsid id, Value *vp);
> +    static JSBool setArgumentsOp(JSContext *cx, JSObject *obj, jsid id, JSBool strict, Value *vp);
> +    static JSBool getArgOp(JSContext *cx, JSObject *obj, jsid id, js::Value *vp);

Kill the js:: in this and the other signatures.

Comment 2

[Jeff Walden [:Waldo]]

...modulo renaming of course.

Comment 3

[Luke Wagner [:luke]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d485f2378abe

Comment 4

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/d485f2378abe