Closed
Bug 757336
Opened 13 years ago
Closed 13 years ago
Cleanup probes handling in CompileScript()
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: n.nethercote, Assigned: n.nethercote)
References
Details
(Whiteboard: [js:t])
Attachments
(1 file)
|
13.60 KB,
patch
|
sfink
:
review+
|
Details | Diff | Splinter Review |
This patch:
- Uses a new one-shot class, ProbesManager, to provide RAII for
handling probes in CompileScript. This allows removal of the goto
statements, and some variable declarations to be moved lower.
- BytecodeEmitter is passed |lineno| instead of tokenStream.getLineno(),
because they're the same thing and |lineno| is simpler.
- TreeContext::init() doesn't need the JSContext* param, so I removed it.
- compileScript{Begin,End}() also had unnecessary params, which I removed.
Attachment #625904 -
Flags: review?(sphink)
|
||
Updated•13 years ago
|
Attachment #625904 -
Flags: review?(sphink) → review+
|
||
Updated•13 years ago
|
Whiteboard: [js:t]
|
|
Assignee | |
Comment 1•13 years ago
|
||
Target Milestone: --- → mozilla15
|
|
||
Comment 2•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•