Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(968)

Unified Diff: lib/src/options.dart

Issue 965033002: add source maps support, fixes #50 (Closed) Base URL: git@github.com:dart-lang/dev_compiler.git@master
Patch Set: format Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: lib/src/options.dart
diff --git a/lib/src/options.dart b/lib/src/options.dart
index 007fe07dce5af3ef1b695aa9021136f03c6e0b02..6e8330002f0467fa1d733a2871368630c34188e4 100644
--- a/lib/src/options.dart
+++ b/lib/src/options.dart
@@ -79,8 +79,15 @@ class RulesOptions extends TypeOptions {
this.relaxedCasts: true, this.ignoreTypes: false});
}
+class JSCodeOptions {
+ /// Whether to emit the source map files.
+ final bool emitSourceMaps;
+
+ JSCodeOptions({this.emitSourceMaps: true});
+}
+
/// General options used by the dev compiler.
-class CompilerOptions implements RulesOptions, ResolverOptions {
+class CompilerOptions implements RulesOptions, ResolverOptions, JSCodeOptions {
/// Whether to check the sdk libraries.
final bool checkSdk;
@@ -180,6 +187,10 @@ class CompilerOptions implements RulesOptions, ResolverOptions {
@override
final bool ignoreTypes;
+ /// Whether to emit the source map files.
+ @override
+ final bool emitSourceMaps;
+
CompilerOptions({this.allowConstCasts: true, this.checkSdk: false,
this.dumpInfo: false, this.dumpInfoFile, this.dumpSrcDir,
this.forceCompile: false, this.formatOutput: false,
@@ -193,7 +204,7 @@ class CompilerOptions implements RulesOptions, ResolverOptions {
this.onlyInferConstsAndFinalFields: false,
this.nonnullableTypes: TypeOptions.NONNULLABLE_TYPES, this.help: false,
this.useMockSdk: false, this.dartSdkPath, this.logLevel: Level.SEVERE,
- this.entryPointFile: null});
+ this.emitSourceMaps: true, this.entryPointFile: null});
}
/// Parses options from the command-line
@@ -233,6 +244,7 @@ CompilerOptions parseOptions(List<String> argv) {
dartSdkPath: sdkPath,
logLevel: Level.LEVELS.firstWhere((Level l) => l.name == levelName,
orElse: () => Level.SEVERE),
+ emitSourceMaps: args['source-maps'],
entryPointFile: args.rest.length == 0 ? null : args.rest.first);
}
@@ -284,6 +296,8 @@ final ArgParser argParser = new ArgParser()
defaultsTo: false)
..addOption('package-paths', help: 'if using the multi-package resolver, '
'the list of directories where to look for packages.', defaultsTo: '')
+ ..addFlag('source-maps',
+ help: 'Whether to emit source map files', defaultsTo: true)
// general options
..addFlag('help', abbr: 'h', help: 'Display this message')

Powered by Google App Engine
This is Rietveld 408576698