Skip to content
This repository has been archived by the owner on Apr 30, 2021. It is now read-only.

Commit

Permalink
Remove --enable-new-task-model flag.
Browse files Browse the repository at this point in the history
BUG=
R=paulberry@google.com

Review URL: https://codereview.chromium.org//1389113002 .
  • Loading branch information
pq committed Oct 6, 2015
1 parent 3e199e4 commit 7dcdbb3
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 36 deletions.
2 changes: 0 additions & 2 deletions lib/src/driver.dart
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,6 @@ class Driver {
// Parse commandline options.
CommandLineOptions options = CommandLineOptions.parse(args);

AnalysisEngine.instance.useTaskModel = options.enableNewTaskModel;

// Cache options of interest to inform analysis.
_setupEnv(options);

Expand Down
4 changes: 0 additions & 4 deletions lib/src/options.dart
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,6 @@ class CommandLineOptions {
/// Whether to display version information
final bool displayVersion;

/// Whether to enable the new task model.
final bool enableNewTaskModel;

/// Whether to enable null-aware operators (DEP 9).
final bool enableNullAwareOperators;

Expand Down Expand Up @@ -111,7 +108,6 @@ class CommandLineOptions {
analysisOptionsFile = args['options'],
disableHints = args['no-hints'],
displayVersion = args['version'],
enableNewTaskModel = args['enable-new-task-model'],
enableNullAwareOperators = args['enable-null-aware-operators'],
enableStrictCallChecks = args['enable-strict-call-checks'],
enableSuperMixins = args['supermixin'],
Expand Down
23 changes: 0 additions & 23 deletions test/driver_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ library analyzer_cli.test.driver;
import 'dart:io';

import 'package:analyzer/plugin/options.dart';
import 'package:analyzer/src/generated/engine.dart';
import 'package:analyzer/src/plugin/plugin_configuration.dart';
import 'package:analyzer_cli/src/bootloader.dart';
import 'package:analyzer_cli/src/driver.dart';
Expand Down Expand Up @@ -36,28 +35,6 @@ main() {
});
});

group('experimental flags', () {
bool savedTaskModelStatus;
setUp(() {
savedTaskModelStatus = AnalysisEngine.instance.useTaskModel;
});

tearDown(() {
AnalysisEngine.instance.useTaskModel = savedTaskModelStatus;
});

test('enable tasks', () {
expect(AnalysisEngine.instance.useTaskModel, savedTaskModelStatus);
Driver driver = new Driver();
try {
driver.start(['--enable-new-task-model', 'test/data/test_file.dart']);
} catch (e) {
//TODO(pq): https://github.com/dart-lang/analyzer_cli/issues/65
}
expect(AnalysisEngine.instance.useTaskModel, true);
});
});

group('exit codes', () {
int savedExitCode;
ExitHandler savedExitHandler;
Expand Down
7 changes: 0 additions & 7 deletions test/options_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ main() {
expect(options.disableHints, isFalse);
expect(options.lints, isFalse);
expect(options.displayVersion, isFalse);
expect(options.enableNewTaskModel, isFalse);
expect(options.enableStrictCallChecks, isFalse);
expect(options.enableSuperMixins, isFalse);
expect(options.enableTypeChecks, isFalse);
Expand Down Expand Up @@ -50,12 +49,6 @@ main() {
expect(options.definedVariables['bar'], isNull);
});

test('enable new task model', () {
CommandLineOptions options = CommandLineOptions
.parse(['--dart-sdk', '.', '--enable-new-task-model', 'foo.dart']);
expect(options.enableNewTaskModel, isTrue);
});

test('enable strict call checks', () {
CommandLineOptions options = CommandLineOptions.parse(
['--dart-sdk', '.', '--enable-strict-call-checks', 'foo.dart']);
Expand Down

0 comments on commit 7dcdbb3

Please sign in to comment.