Closed
Bug 1589905
Opened 6 years ago
Closed 6 years ago
Add BaseScript::nargs() accessor
Categories
(Core :: JavaScript Engine, task, P2)
Core
JavaScript Engine
Tracking
()
RESOLVED
INVALID
People
(Reporter: tcampbell, Assigned: tcampbell)
References
Details
Attachments
(1 obsolete file)
There are several places that access script->function()->nargs() so it is worth having a helper method on BaseScript. This should also handle non-function scripts by returning 0 and existing use cases expect.
|
Assignee | |
Comment 1•6 years ago
|
||
Replace the common pattern of 'script->function()->nargs()' with a
helper method.
Depends on D49838
|
||
Updated•6 years ago
|
Attachment #9102775 -
Attachment description: Bug 1589905 - Add BaseScript::nargs() accessor → Bug 1589905 - Add JSScript::nargs() accessor
|
|
||
Updated•6 years ago
|
Attachment #9102775 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 2•6 years ago
|
||
This is conflicting with a bunch of my other changes and include-order problems. The matter can be revisited once LazyScript/JSScript is merged, but it will look very different so closing as INVALID.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•