diff options
| author | Roman Klauke <romaaan.git@gmail.com> | 2015-05-07 20:27:12 +0200 |
|---|---|---|
| committer | Ben Noordhuis <info@bnoordhuis.nl> | 2015-05-08 19:09:45 +0200 |
| commit | 931a0d463443c68a10c8028f2ebdd01db20da1f9 (patch) | |
| tree | 5babba4596ef3e30d1cbebd3462ebdd5682f6c5a /test | |
| parent | 8bf878d6e5ac0faa3871cddd5dcc423ae4d3d45a (diff) | |
| download | android-node-v8-931a0d463443c68a10c8028f2ebdd01db20da1f9.tar.gz android-node-v8-931a0d463443c68a10c8028f2ebdd01db20da1f9.tar.bz2 android-node-v8-931a0d463443c68a10c8028f2ebdd01db20da1f9.zip | |
src: add type check to v8.setFlagsFromString()
Calling v8.setFlagsFromString with e.g a function as a flag argument
gave no exception or warning that the function call will fail.
There is now an exception if the function gets called with the wrong
flag type (string is required) or that a flag is expected.
Other APIs already provide exceptions if the argument has not the
expected type.
PR-URL: https://github.com/iojs/io.js/pull/1652
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Diffstat (limited to 'test')
| -rw-r--r-- | test/parallel/test-v8-flag-type-check.js | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/test/parallel/test-v8-flag-type-check.js b/test/parallel/test-v8-flag-type-check.js new file mode 100644 index 0000000000..1bb501df43 --- /dev/null +++ b/test/parallel/test-v8-flag-type-check.js @@ -0,0 +1,6 @@ +var common = require('../common'); +var assert = require('assert'); +var v8 = require('v8'); + +assert.throws(function() {v8.setFlagsFromString(1)}, TypeError); +assert.throws(function() {v8.setFlagsFromString()}, TypeError); |