Skip to content

Commit

Permalink
doc: clean up and streamline vm.md examples
Browse files Browse the repository at this point in the history
PR-URL: #31474
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
lundibundi authored and codebytere committed Feb 17, 2020
1 parent 464f4af commit 8f25e51
Showing 1 changed file with 16 additions and 21 deletions.
37 changes: 16 additions & 21 deletions doc/api/vm.md
Original file line number Diff line number Diff line change
Expand Up @@ -175,9 +175,8 @@ for (let i = 0; i < 10; ++i) {
script.runInContext(context);
}

console.log(util.inspect(context));

// { animal: 'cat', count: 12, name: 'kitty' }
console.log(context);
// Prints: { animal: 'cat', count: 12, name: 'kitty' }
```

Using the `timeout` or `breakOnSigint` options will result in new event loops
Expand Down Expand Up @@ -246,9 +245,8 @@ contexts.forEach((context) => {
script.runInNewContext(context);
});

console.log(util.inspect(contexts));

// [{ globalVar: 'set' }, { globalVar: 'set' }, { globalVar: 'set' }]
console.log(contexts);
// Prints: [{ globalVar: 'set' }, { globalVar: 'set' }, { globalVar: 'set' }]
```

### `script.runInThisContext([options])`
Expand Down Expand Up @@ -804,9 +802,11 @@ vm.createContext(context);
vm.runInContext('globalVar *= 2;', context);
console.log(util.inspect(context)); // { globalVar: 2 }
console.log(context);
// Prints: { globalVar: 2 }
console.log(util.inspect(globalVar)); // 3
console.log(global.globalVar);
// Prints: 3
```

If `contextObject` is omitted (or passed explicitly as `undefined`), a new,
Expand Down Expand Up @@ -906,9 +906,8 @@ vm.createContext(contextObject);
for (let i = 0; i < 10; ++i) {
vm.runInContext('globalVar *= 2;', contextObject);
}
console.log(util.inspect(contextObject));
// { globalVar: 1024 }
console.log(contextObject);
// Prints: { globalVar: 1024 }
```
## `vm.runInNewContext(code[, contextObject[, options]])`
Expand Down Expand Up @@ -1003,9 +1002,8 @@ const contextObject = {
};
vm.runInNewContext('count += 1; name = "kitty"', contextObject);
console.log(util.inspect(contextObject));
// { animal: 'cat', count: 3, name: 'kitty' }
console.log(contextObject);
// Prints: { animal: 'cat', count: 3, name: 'kitty' }
```
## `vm.runInThisContext(code[, options])`
Expand Down Expand Up @@ -1075,15 +1073,12 @@ const vm = require('vm');
let localVar = 'initial value';
const vmResult = vm.runInThisContext('localVar = "vm";');
console.log('vmResult:', vmResult);
console.log('localVar:', localVar);
console.log(`vmResult: '${vmResult}', localVar: '${localVar}'`);
// Prints: vmResult: 'vm', localVar: 'initial value'
const evalResult = eval('localVar = "eval";');
console.log('evalResult:', evalResult);
console.log('localVar:', localVar);
// vmResult: 'vm', localVar: 'initial value'
// evalResult: 'eval', localVar: 'eval'
console.log(`evalResult: '${evalResult}', localVar: '${localVar}'`);
// Prints: evalResult: 'eval', localVar: 'eval'
```
Because `vm.runInThisContext()` does not have access to the local scope,
Expand Down

0 comments on commit 8f25e51

Please sign in to comment.