Skip to content

Commit a55af77

Browse files
jasnellMylesBorins
jasnell
authored and
MylesBorins
committed
vm: refactor vm module
Switch to the more efficient module.exports = {} pattern. PR-URL: #11392 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent 818cef8 commit a55af77

8 files changed

+66
-54
lines changed

‎lib/vm.js‎

Lines changed: 34 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -37,17 +37,11 @@ Script.prototype.runInContext = function(contextifiedSandbox, options) {
3737
};
3838

3939
Script.prototype.runInNewContext = function(sandbox, options) {
40-
var context = exports.createContext(sandbox);
40+
var context = createContext(sandbox);
4141
return this.runInContext(context, options);
4242
};
4343

44-
exports.Script = Script;
45-
46-
exports.createScript = function(code, options) {
47-
return new Script(code, options);
48-
};
49-
50-
exports.createContext = function(sandbox) {
44+
function createContext(sandbox) {
5145
if (sandbox === undefined) {
5246
sandbox = {};
5347
} else if (binding.isContext(sandbox)) {
@@ -56,28 +50,11 @@ exports.createContext = function(sandbox) {
5650

5751
binding.makeContext(sandbox);
5852
return sandbox;
59-
};
60-
61-
exports.runInDebugContext = function(code) {
62-
return binding.runInDebugContext(code);
63-
};
64-
65-
exports.runInContext = function(code, contextifiedSandbox, options) {
66-
var script = new Script(code, options);
67-
return script.runInContext(contextifiedSandbox, options);
68-
};
69-
70-
exports.runInNewContext = function(code, sandbox, options) {
71-
var script = new Script(code, options);
72-
return script.runInNewContext(sandbox, options);
73-
};
74-
75-
exports.runInThisContext = function(code, options) {
76-
var script = new Script(code, options);
77-
return script.runInThisContext(options);
78-
};
53+
}
7954

80-
exports.isContext = binding.isContext;
55+
function createScript(code, options) {
56+
return new Script(code, options);
57+
}
8158

8259
// Remove all SIGINT listeners and re-attach them after the wrapped function
8360
// has executed, so that caught SIGINT are handled by the listeners again.
@@ -102,3 +79,31 @@ function sigintHandlersWrap(fn) {
10279
}
10380
}
10481
}
82+
83+
function runInDebugContext(code) {
84+
return binding.runInDebugContext(code);
85+
}
86+
87+
function runInContext(code, contextifiedSandbox, options) {
88+
return createScript(code, options)
89+
.runInContext(contextifiedSandbox, options);
90+
}
91+
92+
function runInNewContext(code, sandbox, options) {
93+
return createScript(code, options).runInNewContext(sandbox, options);
94+
}
95+
96+
function runInThisContext(code, options) {
97+
return createScript(code, options).runInThisContext(options);
98+
}
99+
100+
module.exports = {
101+
Script,
102+
createContext,
103+
createScript,
104+
runInDebugContext,
105+
runInContext,
106+
runInNewContext,
107+
runInThisContext,
108+
isContext: binding.isContext
109+
};

‎test/message/eval_messages.out‎

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@
33
with(this){__filename}
44
^^^^
55
SyntaxError: Strict mode code may not include a with statement
6-
at Object.exports.runInThisContext (vm.js:*)
6+
at createScript (vm.js:*)
7+
at Object.runInThisContext (vm.js:*)
78
at Object.<anonymous> ([eval]-wrapper:*:*)
89
at Module._compile (module.js:*:*)
910
at Immediate.<anonymous> (bootstrap_node.js:*:*)
@@ -15,10 +16,11 @@ SyntaxError: Strict mode code may not include a with statement
1516
[eval]:1
1617
throw new Error("hello")
1718
^
19+
1820
Error: hello
1921
at [eval]:1:7
2022
at ContextifyScript.Script.runInThisContext (vm.js:*)
21-
at Object.exports.runInThisContext (vm.js:*)
23+
at Object.runInThisContext (vm.js:*)
2224
at Object.<anonymous> ([eval]-wrapper:*:*)
2325
at Module._compile (module.js:*:*)
2426
at Immediate.<anonymous> (bootstrap_node.js:*:*)
@@ -28,10 +30,11 @@ Error: hello
2830
[eval]:1
2931
throw new Error("hello")
3032
^
33+
3134
Error: hello
3235
at [eval]:1:7
3336
at ContextifyScript.Script.runInThisContext (vm.js:*)
34-
at Object.exports.runInThisContext (vm.js:*)
37+
at Object.runInThisContext (vm.js:*)
3538
at Object.<anonymous> ([eval]-wrapper:*:*)
3639
at Module._compile (module.js:*:*)
3740
at Immediate.<anonymous> (bootstrap_node.js:*:*)
@@ -42,20 +45,23 @@ Error: hello
4245
[eval]:1
4346
var x = 100; y = x;
4447
^
48+
4549
ReferenceError: y is not defined
4650
at [eval]:1:16
4751
at ContextifyScript.Script.runInThisContext (vm.js:*)
48-
at Object.exports.runInThisContext (vm.js:*)
52+
at Object.runInThisContext (vm.js:*)
4953
at Object.<anonymous> ([eval]-wrapper:*:*)
5054
at Module._compile (module.js:*:*)
5155
at Immediate.<anonymous> (bootstrap_node.js:*:*)
5256
at runCallback (timers.js:*:*)
5357
at tryOnImmediate (timers.js:*:*)
5458
at processImmediate [as _immediateCallback] (timers.js:*:*)
59+
5560
[eval]:1
5661
var ______________________________________________; throw 10
5762
^
5863
10
64+
5965
[eval]:1
6066
var ______________________________________________; throw 10
6167
^

‎test/message/stdin_messages.out‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
[stdin]
2-
32
[stdin]:1
43
with(this){__filename}
54
^^^^
65
SyntaxError: Strict mode code may not include a with statement
7-
at Object.exports.runInThisContext (vm.js:*)
6+
at createScript (vm.js:*)
7+
at Object.runInThisContext (vm.js:*)
88
at Object.<anonymous> ([stdin]-wrapper:*:*)
99
at Module._compile (module.js:*:*)
1010
at Immediate.<anonymous> (bootstrap_node.js:*:*)
@@ -13,43 +13,43 @@ SyntaxError: Strict mode code may not include a with statement
1313
at processImmediate [as _immediateCallback] (timers.js:*:*)
1414
42
1515
42
16-
1716
[stdin]:1
1817
throw new Error("hello")
1918
^
19+
2020
Error: hello
21-
at [stdin]:1:*
21+
at [stdin]:1:7
2222
at ContextifyScript.Script.runInThisContext (vm.js:*)
23-
at Object.exports.runInThisContext (vm.js:*)
23+
at Object.runInThisContext (vm.js:*)
2424
at Object.<anonymous> ([stdin]-wrapper:*:*)
2525
at Module._compile (module.js:*:*)
2626
at Immediate.<anonymous> (bootstrap_node.js:*:*)
2727
at runCallback (timers.js:*:*)
2828
at tryOnImmediate (timers.js:*:*)
2929
at processImmediate [as _immediateCallback] (timers.js:*:*)
30-
3130
[stdin]:1
3231
throw new Error("hello")
3332
^
33+
3434
Error: hello
3535
at [stdin]:1:*
3636
at ContextifyScript.Script.runInThisContext (vm.js:*)
37-
at Object.exports.runInThisContext (vm.js:*)
37+
at Object.runInThisContext (vm.js:*)
3838
at Object.<anonymous> ([stdin]-wrapper:*:*)
3939
at Module._compile (module.js:*:*)
4040
at Immediate.<anonymous> (bootstrap_node.js:*:*)
4141
at runCallback (timers.js:*:*)
4242
at tryOnImmediate (timers.js:*:*)
4343
at processImmediate [as _immediateCallback] (timers.js:*:*)
4444
100
45-
4645
[stdin]:1
4746
var x = 100; y = x;
4847
^
48+
4949
ReferenceError: y is not defined
5050
at [stdin]:1:16
5151
at ContextifyScript.Script.runInThisContext (vm.js:*)
52-
at Object.exports.runInThisContext (vm.js:*)
52+
at Object.runInThisContext (vm.js:*)
5353
at Object.<anonymous> ([stdin]-wrapper:*:*)
5454
at Module._compile (module.js:*:*)
5555
at Immediate.<anonymous> (bootstrap_node.js:*:*)

‎test/message/undefined_reference_in_new_context.out‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
before
2-
32
evalmachine.<anonymous>:1
43
foo.bar = 5;
54
^
5+
66
ReferenceError: foo is not defined
77
at evalmachine.<anonymous>:1:1
88
at ContextifyScript.Script.runInContext (vm.js:*)
99
at ContextifyScript.Script.runInNewContext (vm.js:*)
10-
at Object.exports.runInNewContext (vm.js:*)
10+
at Object.runInNewContext (vm.js:*)
1111
at Object.<anonymous> (*test*message*undefined_reference_in_new_context.js:*)
1212
at Module._compile (module.js:*)
1313
at *..js (module.js:*)

‎test/message/vm_display_runtime_error.out‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
beginning
2-
32
test.vm:1
43
throw new Error("boo!")
54
^
5+
66
Error: boo!
77
at test.vm:1:7
88
at ContextifyScript.Script.runInThisContext (vm.js:*)
9-
at Object.exports.runInThisContext (vm.js:*)
9+
at Object.runInThisContext (vm.js:*)
1010
at Object.<anonymous> (*test*message*vm_display_runtime_error.js:*)
1111
at Module._compile (module.js:*)
1212
at Object.Module._extensions..js (module.js:*)

‎test/message/vm_display_syntax_error.out‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
beginning
2-
32
foo.vm:1
43
var 4;
54
^
65
SyntaxError: Unexpected number
7-
at Object.exports.runInThisContext (vm.js:*)
6+
at createScript (vm.js:*)
7+
at Object.runInThisContext (vm.js:*)
88
at Object.<anonymous> (*test*message*vm_display_syntax_error.js:*)
99
at Module._compile (module.js:*)
1010
at Object.Module._extensions..js (module.js:*)
@@ -13,12 +13,12 @@ SyntaxError: Unexpected number
1313
at Function.Module._load (module.js:*)
1414
at Module.runMain (module.js:*)
1515
at run (bootstrap_node.js:*)
16-
at startup (bootstrap_node.js:*)
1716
test.vm:1
1817
var 5;
1918
^
2019
SyntaxError: Unexpected number
21-
at Object.exports.runInThisContext (vm.js:*)
20+
at createScript (vm.js:*)
21+
at Object.runInThisContext (vm.js:*)
2222
at Object.<anonymous> (*test*message*vm_display_syntax_error.js:*)
2323
at Module._compile (module.js:*)
2424
at Object.Module._extensions..js (module.js:*)
@@ -27,4 +27,3 @@ SyntaxError: Unexpected number
2727
at Function.Module._load (module.js:*)
2828
at Module.runMain (module.js:*)
2929
at run (bootstrap_node.js:*)
30-
at startup (bootstrap_node.js:*)

‎test/message/vm_dont_display_runtime_error.out‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,11 @@ middle
33
test.vm:1
44
throw new Error("boo!")
55
^
6+
67
Error: boo!
78
at test.vm:1:7
89
at ContextifyScript.Script.runInThisContext (vm.js:*)
9-
at Object.exports.runInThisContext (vm.js:*)
10+
at Object.runInThisContext (vm.js:*)
1011
at Object.<anonymous> (*test*message*vm_dont_display_runtime_error.js:*)
1112
at Module._compile (module.js:*)
1213
at Object.Module._extensions..js (module.js:*)

‎test/message/vm_dont_display_syntax_error.out‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,10 @@ middle
33
test.vm:1
44
var 5;
55
^
6+
67
SyntaxError: Unexpected number
7-
at Object.exports.runInThisContext (vm.js:*)
8+
at createScript (vm.js:*)
9+
at Object.runInThisContext (vm.js:*)
810
at Object.<anonymous> (*test*message*vm_dont_display_syntax_error.js:*)
911
at Module._compile (module.js:*)
1012
at Object.Module._extensions..js (module.js:*)
@@ -13,4 +15,3 @@ SyntaxError: Unexpected number
1315
at Function.Module._load (module.js:*)
1416
at Module.runMain (module.js:*)
1517
at run (bootstrap_node.js:*)
16-
at startup (bootstrap_node.js:*)

0 commit comments

Comments
 (0)