Skip to content

Commit

Permalink
test: remove harmony flags
Browse files Browse the repository at this point in the history
v8 6.8 supports all removed flags. For example for BigInt.
  • Loading branch information
BridgeAR committed Aug 13, 2018
1 parent 1f32cca commit 0450484
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 17 deletions.
10 changes: 2 additions & 8 deletions doc/api/util.md
Original file line number Diff line number Diff line change
Expand Up @@ -1021,10 +1021,7 @@ added: v10.0.0
* `value` {any}
* Returns: {boolean}

Returns `true` if the value is a `BigInt64Array` instance. The
`--harmony-bigint` command line flag is required in order to use the
`BigInt64Array` type, but it is not required in order to use
`isBigInt64Array()`.
Returns `true` if the value is a `BigInt64Array` instance.

For example:

Expand All @@ -1041,10 +1038,7 @@ added: v10.0.0
* `value` {any}
* Returns: {boolean}

Returns `true` if the value is a `BigUint64Array` instance. The
`--harmony-bigint` command line flag is required in order to use the
`BigUint64Array` type, but it is not required in order to use
`isBigUint64Array()`.
Returns `true` if the value is a `BigUint64Array` instance.

For example:

Expand Down
4 changes: 0 additions & 4 deletions src/node.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2958,10 +2958,6 @@ static void ParseArgs(int* argc,
config_preserve_symlinks_main = true;
} else if (strcmp(arg, "--experimental-modules") == 0) {
config_experimental_modules = true;
new_v8_argv[new_v8_argc] = "--harmony-dynamic-import";
new_v8_argc += 1;
new_v8_argv[new_v8_argc] = "--harmony-import-meta";
new_v8_argc += 1;
} else if (strcmp(arg, "--experimental-vm-modules") == 0) {
config_experimental_vm_modules = true;
} else if (strcmp(arg, "--experimental-worker") == 0) {
Expand Down
2 changes: 0 additions & 2 deletions test/parallel/test-util-inspect-bigint.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
'use strict';

// Flags: --harmony-bigint

require('../common');
const assert = require('assert');

Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-util-types.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Flags: --harmony-bigint --experimental-vm-modules
// Flags: --experimental-vm-modules
'use strict';
require('../common');
const fixtures = require('../common/fixtures');
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-vm-module-dynamic-import.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

// Flags: --experimental-vm-modules --experimental-modules --harmony-dynamic-import
// Flags: --experimental-vm-modules --experimental-modules

const common = require('../common');

Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-vm-module-import-meta.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
'use strict';

// Flags: --experimental-vm-modules --harmony-import-meta
// Flags: --experimental-vm-modules

const common = require('../common');
const assert = require('assert');
Expand Down

0 comments on commit 0450484

Please sign in to comment.