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

Unified Diff: compiler/java/com/google/dart/compiler/CommandLineOptions.java

Issue 9702034: Removes dartc reliance on its own libraries, now can be targeted at any implementation's libraries (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: junit tests fixed Created 8 years, 9 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
« no previous file with comments | « compiler/eclipse.workspace/dartc/.project ('k') | compiler/java/com/google/dart/compiler/DartCompiler.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: compiler/java/com/google/dart/compiler/CommandLineOptions.java
diff --git a/compiler/java/com/google/dart/compiler/CommandLineOptions.java b/compiler/java/com/google/dart/compiler/CommandLineOptions.java
index 3a4dea3c931e29e0d8fc695df1360648e11d132c..cdc7394515d03a25c9e2f74ad01e959a6680b77f 100644
--- a/compiler/java/com/google/dart/compiler/CommandLineOptions.java
+++ b/compiler/java/com/google/dart/compiler/CommandLineOptions.java
@@ -79,8 +79,7 @@ public class CommandLineOptions {
usage = "Enable incremental compilation")
private boolean incremental = false;
- // TODO(zundel): -out is for backward compatibility until scripts are updated
- @Option(name = "--work", aliases = { "-out" },
+ @Option(name = "--work",
usage = "Directory to receive compiler output\n for future incremental builds")
private File workDirectory = new File("out");
@@ -104,6 +103,17 @@ public class CommandLineOptions {
usage = "Treat non-type warnings as fatal")
private boolean warningsAreFatal = false;
+ @Option(name = "--implementation-name",
+ usage = "Implementation to analyze (e.g. dartium, vm, dart2js, frog)")
+ private String implementationName = SystemLibraryManager.DEFAULT_IMPLEMENTATION;
+
+ @Option(name = "--dart-sdk",
+ usage = "Path to dart sdk. (system property com.google.dart.sdk)")
+ private String dartSdkPath = SystemLibraryManager.DEFAULT_SDK_PATH;
+
+ @Option(name = "--show-sdk-warnings", usage = "show warnings from SDK source")
+ private boolean showSdkWarnings = false;
+
@Argument
private final List<String> sourceFiles = new ArrayList<String>();
@@ -121,6 +131,21 @@ public class CommandLineOptions {
return jvmMetricDetail + ":" + jvmMetricFormat + ":" + jvmMetricType;
}
+ public String getImplementationName() {
+ return implementationName;
+ }
+
+ public String getDartSdkPath() {
+ return dartSdkPath;
+ }
+
+ /**
+ * Returns whether warnings from SDK files should be suppressed.
+ */
+ public boolean suppressSdkWarnings() {
+ return !showSdkWarnings;
+ }
+
/**
* Returns the list of files passed to the compiler.
*/
« no previous file with comments | « compiler/eclipse.workspace/dartc/.project ('k') | compiler/java/com/google/dart/compiler/DartCompiler.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698